Skip to content
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

gh-92412: Clarifying the documentation on library/syslog #92587

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Gaasmann
Copy link

@Gaasmann Gaasmann commented May 9, 2022

Clarifying the "Changed in version 3.2" paragraph for syslog.openlog as
the wording could confuse people about ident being a mandatory argument
with a default value.

@cpython-cla-bot
Copy link

@cpython-cla-bot cpython-cla-bot bot commented May 9, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented May 9, 2022

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented May 9, 2022

Every change to Python requires a NEWS entry.

Please, add it using the blurb_it Web app or the blurb command-line tool.

Clarifying the "Changed in version 3.2" paragraph for syslog.openlog as
the wording could confuse people about *ident* being a mandatory argument
with a default value.
@Gaasmann Gaasmann changed the title gh-92412: Clarifying the documentaion on library/syslog gh-92412: Clarifying the documentation on library/syslog May 9, 2022
@Gaasmann
Copy link
Author

@Gaasmann Gaasmann commented May 9, 2022

@rhettinger Thank you for approving my first pull request :-)

@Gaasmann
Copy link
Author

@Gaasmann Gaasmann commented May 11, 2022

Hello @rhettinger, the PR is ready to be merged. Let me know if there is anything I need to do, this is pretty new to me :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge docs skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants