Issues Show Unassigned Show All Search
Login Remember me? Register Lost your login?
Help Roundup docs
Created on 2017-04-25.14:47:35 by jendrik, last changed by jendrik.
Done :-)
Yes, please merge.
Thanks for your comments! I have taken care of them and checked that the help and txt2tags output hasn't changed. Can I go ahead and merge this?
I had a brief look at some, but not all of the code and left a few comments. Before merging, I think we should verify that the before/after output of the option parser's "dump all options" features (for humans and for txt2tags) remain the same.
I'll have a look. If you're impatient, feel free to switch the position in the queue with one of your other entries. ;-)
Thanks, Silvan! I'm done taking care of the comments. Malte, if you like, you could have a look at the pull request.
I'm done with the first round of comments.
I have prepared a pull request at https://bitbucket.org/jendrikseipp/downward/pull-requests/67 and added the issue to the reviewing queue.
While trying to understand the option parser, I will bring the option parser code closer to our coding conventions.