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

Add people's last words (easy PR) #1

Open
CodeWritingCow opened this issue Oct 30, 2017 · 6 comments
Open

Add people's last words (easy PR) #1

CodeWritingCow opened this issue Oct 30, 2017 · 6 comments

Comments

@CodeWritingCow
Copy link
Owner

CodeWritingCow commented Oct 30, 2017

Add people's last words, whether famous or obscure. Just follow the specs in README.md!

@NeroAzure
Copy link
Contributor

NeroAzure commented Oct 30, 2017

how about lastnames that are separated by a hyphen? like Bismarck-Schönhausen? wouldn't it screw with your file naming convention?

@spanishharlem
Copy link
Contributor

How about naming files with people such as Pope John Paul II?

@TwoRice
Copy link
Contributor

TwoRice commented Oct 30, 2017

How should the naming work for people with stage names, nicknames or shortened names that differ from their birth certificate names?

mailtobash referenced this issue in mailtobash/their-last-words Oct 31, 2017
mjzone referenced this issue in mjzone/their-last-words Oct 31, 2017
nilupull pushed a commit to nilupull/their-last-words that referenced this issue Oct 31, 2017
@CodeWritingCow
Copy link
Owner Author

@NeroAzure that's a good question. I'm leaning toward using the hyphenated last name, ex. bismarck-schönhausen-josef.json. What do you think?

@CodeWritingCow
Copy link
Owner Author

@Nalapl3 and @TwoRice, let's go with the names that they were best known by. A couple example:

  • pope-john-paul-ii.json.
  • Same with stage names. Everyone knows monroe-marilyn.json. Only a few would recognize mortenson-norma-jeane.json.
    I'll update README.md accordingly. These questions never occurred to me until you asked. Thanks guys!

CodeWritingCow added a commit that referenced this issue Oct 31, 2017
Issue #1 Add Arthur conan doyle name
@knighthedspi
Copy link

I create PR28 for Alexander the Great
Please review

CodeWritingCow added a commit that referenced this issue Nov 2, 2017
CodeWritingCow added a commit that referenced this issue Nov 7, 2017
Also add note to Napoloean file.
CodeWritingCow added a commit that referenced this issue Nov 8, 2017
CodeWritingCow added a commit that referenced this issue Dec 27, 2017
CodeWritingCow added a commit that referenced this issue Dec 31, 2017
CodeWritingCow added a commit that referenced this issue Jan 2, 2018
CodeWritingCow added a commit that referenced this issue Jan 2, 2018
CodeWritingCow added a commit that referenced this issue Mar 8, 2018
CodeWritingCow added a commit that referenced this issue May 6, 2018
CodeWritingCow added a commit that referenced this issue May 19, 2018
Add Christine Chubbuck, Empress Josephine and Dominick Napolitano. Edit
Christopher Columbus. Issue #1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants