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 support for github usernames in author lines #974

Merged
merged 2 commits into from
Apr 6, 2018

Conversation

Arachnid
Copy link
Contributor

@Arachnid Arachnid commented Apr 5, 2018

No description provided.

@Arachnid Arachnid requested a review from nicksavers April 5, 2018 12:45
@nicksavers
Copy link
Contributor

It should be pretty obvious for editors that this refers to a GitHub username, but to any other reader it might look like a Twitter handle or another username. Is there a way to make this more obvious? Is it really something to be concerned about?

@Arachnid
Copy link
Contributor Author

Arachnid commented Apr 6, 2018

Well, on the generated site it links to the GitHub page.

@nicksavers
Copy link
Contributor

Examples already exist

I suppose it has already been established as an acceptable contact method and this pull request would just make use of it.

@nicksavers nicksavers merged commit 5b37c1d into ethereum:master Apr 6, 2018
@Arachnid Arachnid deleted the logins branch April 7, 2018 06:46
@4abdulkholik
Copy link

ok

Arachnid added a commit to Arachnid/EIPs that referenced this pull request May 2, 2018
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.

3 participants