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.
After #425, there are 9 files in
humans/
relating to names, which is a substantial chunk of the folder. I've taken the liberty of reorganizing them in this PR rather than creating an issue first to discuss the changes, with anames/
sub-folder.Pros:
data/games/rpg/
anddata/words/literature/
).humans/
).Cons:
git-submodule
, for example).As per 3, I've taken the liberty to rename the files for a more elegant and cohesive naming scheme. for example:
firstNanes.json
andlastNames.json
tousa.json
andusaSurnames.json
norwayFirstNamesBoys.json
tonorwayMale.json
I hope the benefits out-weight the compatibility issue, especially as it's safe to expect more name-related files to be added again in the future but I understand this might be a concern for some projects.
Here's a simpler view of the folder changes: https://github.com/tukkek/corpora/tree/names-folder/data/humans/names