added support for multiple dictionaries #9
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
Following your comment on Reddit, this is my attempt at making a pull request with the changes I've made at zippoxer/petname.
This pull request adds support for multiple dictionaries by introducing the
Dict
interface, which is implemented by packages under thedict
folder.The usage of the
Generate
method has also changed, and it now takes aDict
as it's first argument:I should note that this pull request might break some of the scripts under the
debian
folder, since I didn't yet take a close look in there.I'm taking the old repository (zippoxer/petname) down, as this fork provides the same functionality. I hope this would make things right between us.
Cheers :-)