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

renamed basic-bag to basic-bag-md5 and added basic bags for other sta… #10

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

Conversation

jscancella
Copy link
Contributor

…ndard algorithm names

@acdha
Copy link
Member

acdha commented Aug 1, 2018

What's the rationale for having this as separate bags rather than adding other algorithms to the existing bag? Just to keep error logging simple?

@jscancella
Copy link
Contributor Author

jscancella commented Aug 1, 2018

@acdha to be consistent. I originally intended to also create new bags with the other "standard" algorithms supported (SHA-1, SHA-256, SHA-512) for all the versions (not just 0.97) but I got sidetracked and haven't been able to commit those new bags

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.

2 participants