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-48511: Make pwd and grp expose inner errors #102223

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

simaoafonso-pwt
Copy link

@simaoafonso-pwt simaoafonso-pwt commented Feb 24, 2023

Distinguish between the case when the user/group does not exist and error conditions.
If the user does not exit, raise KeyError as now, for errors raise OSError.

Based on arhadthedev@46c34b6

Distinguish
between the case when the user/group does not exist and error
conditions.
If the user does not exit, raise KeyError as  now, for errors raise OSError.
@cpython-cla-bot
Copy link

cpython-cla-bot bot commented Feb 24, 2023

All commit authors signed the Contributor License Agreement.
CLA signed

@arhadthedev arhadthedev added the extension-modules C modules in the Modules dir label Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review extension-modules C modules in the Modules dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants