Skip to content

bpo-37908: Add an example of ArgumentParser.exit()#15455

Merged
matrixise merged 11 commits intopython:masterfrom
shihai1991:bpo_37908
Sep 12, 2019
Merged

bpo-37908: Add an example of ArgumentParser.exit()#15455
matrixise merged 11 commits intopython:masterfrom
shihai1991:bpo_37908

Conversation

@shihai1991
Copy link
Copy Markdown
Member

@shihai1991 shihai1991 commented Aug 24, 2019 •
edited by bedevere-bot
Loading

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Aug 24, 2019
@shihai1991 shihai1991 changed the title bpo-37908: Add an example of ArgumentParser.Exit() bpo-37908: Add an example of ArgumentParser.exit() Aug 24, 2019
brandtbucher
brandtbucher requested changes Aug 24, 2019
View reviewed changes
@brandtbucher brandtbucher mentioned this pull request Aug 24, 2019
Closed
shihai1991 and others added 2 commits August 25, 2019 16:38
@shihai1991 @brandtbucher
Update Doc/library/argparse.rst
5be636e 
Co-Authored-By: Brandt Bucher <brandtbucher@gmail.com>
@shihai1991 @brandtbucher
Update Doc/library/argparse.rst
681ce14 
Co-Authored-By: Brandt Bucher <brandtbucher@gmail.com>
@shihai1991
Copy link
Copy Markdown
Member Author

shihai1991 commented Sep 3, 2019

cc @rhettinger

matrixise
matrixise requested changes Sep 11, 2019
View reviewed changes
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Sep 11, 2019

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@matrixise matrixise self-assigned this Sep 12, 2019
shihai1991
shihai1991 commented Sep 12, 2019
View reviewed changes
Doc/library/argparse.rst Outdated
this method to handle these steps differently::

import sys

Copy link
Copy Markdown
Member Author

@shihai1991 shihai1991 Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two lines of space?

Copy link
Copy Markdown
Member

@matrixise matrixise Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just one line,

import sys

class ErrorCatchingArgumentParser(argparse.ArgumentParser):
    pass

Copy link
Copy Markdown
Member Author

@shihai1991 shihai1991 Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I check some other examples in rst, single line is fine.

Copy link
Copy Markdown
Member

@JulienPalard JulienPalard Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yet the rst indentation is wrong (single space)

Copy link
Copy Markdown
Member Author

@shihai1991 shihai1991 Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JulienPalard And i think it is confusing some users too :(.
I found some code examples in rst using >>> and the others don't using >>> like this example. it is confusing too(i don't have time to look the details.)

Copy link
Copy Markdown
Member

@JulienPalard JulienPalard Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blocks with '>>>' are here to demostrate an interactive session (when the responses are important), like:

>>> 1.2 - 1.0
0.19999999999999996

The ones with no '>>>' are to demonstrate code (implementation of classes, functions, ...) when there is typically no result, like:

def foo(x, y):
    return 5 * x - 3 * y

Copy link
Copy Markdown
Member Author

@shihai1991 shihai1991 Sep 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, thanks, Julien Palard. First time to know the difference.

@shihai1991
Copy link
Copy Markdown
Member Author

shihai1991 commented Sep 12, 2019

@matrixise oh, forgive me to cover your patch, i reverse to your patch.

@matrixise
Copy link
Copy Markdown
Member

matrixise commented Sep 12, 2019

Please, stop to change the code, because your PR was ok, and Travis restart a new job at each push. and Travis is not quick ;-)

@shihai1991
Copy link
Copy Markdown
Member Author

shihai1991 commented Sep 12, 2019

Please, stop to change the code, because your PR was ok, and Travis restart a new job at each push. and Travis is not quick ;-)

Forgive me. I stop changing the code now.

def exit(self, status=0, message=None) lack a colon at the end, i updated it.

@matrixise
Copy link
Copy Markdown
Member

matrixise commented Sep 12, 2019

Nice, thank you

@matrixise matrixise merged commit b1a2abd into python:master Sep 12, 2019
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Sep 12, 2019

Thanks @shihai1991 for the PR, and @matrixise for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Sep 12, 2019

GH-16065 is a backport of this pull request to the 3.8 branch.

@shihai1991 shihai1991 deleted the bpo_37908 branch September 12, 2019 15:41
miss-islington added a commit that referenced this pull request Sep 12, 2019
@miss-islington @shihai1991
bpo-37908: Add an example of ArgumentParser.exit() (GH-15455)
6dc3e61 
Co-Authored-By: Brandt Bucher <brandtbucher@gmail.com>
(cherry picked from commit b1a2abd)

Co-authored-by: Hai Shi <shihai1992@gmail.com>
@shihai1991
Copy link
Copy Markdown
Member Author

shihai1991 commented Sep 12, 2019

Thanks a million, guys. Looks all my PR which waiting core developer review have been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JulienPalard JulienPalard JulienPalard left review comments

@brandtbucher brandtbucher brandtbucher requested changes

@matrixise matrixise matrixise approved these changes

Assignees

@matrixise matrixise

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@shihai1991 @bedevere-bot @matrixise @miss-islington @JulienPalard @brandtbucher @the-knights-who-say-ni