-
Notifications
You must be signed in to change notification settings - Fork 24
More on tutorial #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More on tutorial #23
Conversation
@olleolleolle, will you be looking at this? |
@nobu, can you take a look at this PR? (I have a biggish block of RDoc about parsing that's waiting for this PR to clear.) Thanks. |
@BurdetteLamar It's been a bit hectic, here and I've been far from computer screens when not at work. Soon! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the additional context added by this PR!
The parts which say "these work like these other methods, but return differently" is especially good for the careful reader.
Added comma, as suggested.
Thanks, @olleolleolle, for your careful reviews. |
- Removed a largish block of repeated text. - Added sections "Top List and Base List" and "Methods for Defining Options" (on, define, etc.). - Linked from class OptionParser doc to the tutorial. ruby/optparse@7f3195b9db
I'm not positive that we need the two sections, but it took be some work to learn and write, so let's consider them.