-
Notifications
You must be signed in to change notification settings - Fork 18.6k
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 more amazing women to the names generator. #15218
Conversation
// Satyendra Nath Bose - He provided the foundation for Bose–Einstein statistics and the theory of the Bose–Einstein condensate. - https://en.wikipedia.org/wiki/Satyendra_Nath_Bose | ||
"bose", | ||
|
||
// Evelyn Boyd Granville - Swas one of the first African-American women to receive a Ph.D. in mathematics; she earned it in 1949 from Yale University. https://en.wikipedia.org/wiki/Evelyn_Boyd_Granville |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
woman
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/Swas one/One
031ac3b
to
3af8e06
Compare
👍 |
@@ -149,6 +165,12 @@ var ( | |||
// Leonardo Da Vinci invented too many things to list here. https://en.wikipedia.org/wiki/Leonardo_da_Vinci. | |||
"davinci", | |||
|
|||
// Donna Dubinsky, CEO and co-founder of Palm, Inc. https://en.wikipedia.org/wiki/Donna_Dubinsky | |||
"dubinsky", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will probably not make me popular; is co-founding a company notable enough to make it in this list?
(Read the linked Wikipedia page; there are some notable things in there, so possibly with a better description)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Read through the list again; possibly I'm too strict, just want to guard against companies wanting to add founders to the list, just because they founded a company.
Signed-off-by: David Calavera <david.calavera@gmail.com>
3af8e06
to
85a0f56
Compare
LGTM |
1 similar comment
LGTM |
Add more amazing women to the names generator.
Did you know that we have 62 men in the names generator but only 33 women?
This change makes the number of men and women equal.
Signed-off-by: David Calavera david.calavera@gmail.com