Skip to content

Ndbm move#299

Merged
benjaminp merged 4 commits into
benjaminp:masterfrom
benbariteau:ndbm_move
Nov 5, 2019
Merged

Ndbm move#299
benjaminp merged 4 commits into
benjaminp:masterfrom
benbariteau:ndbm_move

Conversation

@benbariteau

Copy link
Copy Markdown
Contributor

refs #298

@graingert

graingert commented Oct 3, 2019

Copy link
Copy Markdown
Contributor

@graingert

Copy link
Copy Markdown
Contributor

@benbariteau

benbariteau commented Oct 4, 2019

Copy link
Copy Markdown
Contributor Author

There's a (not finished) PR for one of those. #180

@benbariteau

Copy link
Copy Markdown
Contributor Author

If you want me to add more moves, I'm happy to do so, but this is the one I really care about.

@graingert

Copy link
Copy Markdown
Contributor

I'd like to see a comprehensive PR will all the moves added, but I'm not the maintainer and I believe this package is now unmaintained

@benjaminp ^

@benjaminp

Copy link
Copy Markdown
Owner

The six.moves name should be dbm_ndbm.

@benjaminp benjaminp mentioned this pull request Nov 5, 2019
@benbariteau

Copy link
Copy Markdown
Contributor Author

Should I add the other dbm moves as part of this branch?

@benjaminp benjaminp merged commit 1f2f571 into benjaminp:master Nov 5, 2019
@benjaminp

Copy link
Copy Markdown
Owner

Which are those?

@benbariteau

Copy link
Copy Markdown
Contributor Author

anydbm and whichdb

@benbariteau

Copy link
Copy Markdown
Contributor Author

ah and dumbdbm

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants