Skip to content

bpo-40563: Support pathlike objects on dbm/shelve#21849

Merged
serhiy-storchaka merged 13 commits intopython:mainfrom
audeoudh:bpo-40563
Sep 10, 2021
Merged

bpo-40563: Support pathlike objects on dbm/shelve#21849
serhiy-storchaka merged 13 commits intopython:mainfrom
audeoudh:bpo-40563

Conversation

@audeoudh
Copy link
Contributor

@audeoudh audeoudh commented Aug 12, 2020

Refers to https://bugs.python.org/issue40563.

This work continues @hakancelik96's work from #20274.

https://bugs.python.org/issue40563

@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@audeoudh

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

@ethanfurman
Copy link
Member

The CLA has been signed (according to the knight).

@audeoudh
Copy link
Contributor Author

audeoudh commented Sep 8, 2021

It was a long time I did not check this PR. I hope I did not forget something in the meantime.

I rebased the work on current main branch. I took @serhiy-storchaka's review into account.

I suppose I should squash before the merge, shouldn't I? Just let me know, I'll do it.

@erlend-aasland
Copy link
Contributor

I rebased the work on current main branch.

Force-pushing does not play well with GitHub and reviews. I would suggest to use git merge --no-ff main instead of git rebase main, as the review history is kept intact.

I suppose I should squash before the merge, shouldn't I? Just let me know, I'll do it.

No need; all PR's are squashed upon merging :)

Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

The code LGTM. Please fix test_whichdb_ndbm and add tests for bytes paths.

@audeoudh
Copy link
Contributor Author

audeoudh commented Sep 9, 2021

Force-pushing does not play well with GitHub and reviews. I would suggest to use git merge --no-ff main instead of git rebase main, as the review history is kept intact.

True for GitHub reviews. But merge --no-ff without rebase does not play well with git log itself.

But if all PR's are squased upon merging, it does not need to be rebased. Thanks for the tip.

Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM.

@serhiy-storchaka serhiy-storchaka added the type-feature A feature request or enhancement label Sep 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-feature A feature request or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants