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

Use em dash #226

Merged
merged 1 commit into from
Apr 13, 2017
Merged

Use em dash #226

merged 1 commit into from
Apr 13, 2017

Conversation

dcow
Copy link
Contributor

@dcow dcow commented Feb 5, 2017

Pretentious quotes should be pretentious. (:

Copy link
Collaborator

@rjrjr rjrjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this important PR! Please remember to sign our CLA so that I can merge it.

https://github.com/square/flow/blob/master/CONTRIBUTING.md

README.md Outdated
@@ -1,10 +1,10 @@
# Flow

_"Name-giving will be the foundation of our science."_ - Linnaeus
_"Name-giving will be the foundation of our science."_ —Linnaeus
Copy link
Collaborator

@rjrjr rjrjr Apr 12, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be pretentious, yes, but should also have consistent spacing. There should either be no leading space before the dash, or there should be a trailing one. I prefer the latter.

Pretentious quotes should be pretentious. (:
@dcow
Copy link
Contributor Author

dcow commented Apr 13, 2017

Just having some fun! Here's a good read: https://en.wikipedia.org/wiki/Dash#En_dash_versus_em_dash

@dcow
Copy link
Contributor Author

dcow commented Apr 13, 2017

I updated based on your requested changes and signed the agreement, btw.

@rjrjr rjrjr merged commit 6971db9 into square:master Apr 13, 2017
@rjrjr
Copy link
Collaborator

rjrjr commented Apr 13, 2017

I love a good em-dash — I use them all the time.

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.

None yet

2 participants