bpo-9938: Add optional keyword argument exit_on_error to argparse.ArgumentParser by shihai1991 · Pull Request #15362 · python/cpython (original) (raw)

Choose a reason for hiding this comment

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

@shihai1991

the desc looks good to me. But why the title need add the bpo link now? the format of title have been changed?

I may have been a bit unclear, apologies. I just included the bpo part so that you wouldn't accidentally remove it when editing the title and I didn't intend to include the link. It looks good now.

Thanks for making the recommended changes.

I have a couple of minor grammar suggestions, but once those are added, everything documentation-related looks good as far as I can tell. (: