Skip to content

bpo-41115: Modified src to raise rather Unicode{Encode, Decode}Error rather than plain UnicodeError #21170

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

Closed
wants to merge 2 commits into from

Conversation

utkarsh261
Copy link

@utkarsh261 utkarsh261 commented Jun 26, 2020

@srinivasreddy
Copy link
Contributor

This is blatant copy of my PR #21165 . Why do you want to do it? @utkarsh261

@utkarsh261
Copy link
Author

utkarsh261 commented Jun 26, 2020

@srinivasreddy These were the most logical changes, standard error messages, which I used from already existing code, they were already there, i just had to edit them. What part of your "work" do you think i copied?
Was sending this PR to get familiar to the process mostly, if you think your work was copied, reviewers will notice that, please dont panic.

@utkarsh261
Copy link
Author

Closing this as @srinivasreddy already submitted a PR and I missed it earlier even though it looks like similar code.
Thanks.

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

Successfully merging this pull request may close these issues.

5 participants