Skip to content

Add Github usernames#990

Merged
Arachnid merged 6 commits intoethereum:masterfrom
jamesray1:patch-20
Apr 11, 2018
Merged

Add Github usernames#990
Arachnid merged 6 commits intoethereum:masterfrom
jamesray1:patch-20

Conversation

@jamesray1
Copy link
Contributor

@jamesray1 jamesray1 commented Apr 10, 2018 •
edited
Loading

Added Github usernames and content from the original PR.

@jamesray1
Copy link
Contributor Author

jamesray1 commented Apr 10, 2018

What's the format for adding Github usernames in triangular brackets?

@5chdn
Copy link
Contributor

5chdn commented Apr 10, 2018

It appears you are not only adding GitHub usernames. What are the other additions about?

@nicksavers
Copy link
Contributor

nicksavers commented Apr 10, 2018

@jamesray1 Look at https://github.com/ethereum/EIPs/blob/master/EIPS/eip-191.md for an example

@jamesray1
Copy link
Contributor Author

jamesray1 commented Apr 10, 2018

It appears you are not only adding GitHub usernames. What are the other additions about?

Yeah, I forgot to add that to the commit summaries. I was moving content from the PR to the EIP.

@jamesray1
Copy link
Contributor Author

jamesray1 commented Apr 10, 2018

Look at https://github.com/ethereum/EIPs/blob/master/EIPS/eip-191.md for an example

Yeah I thought it might be a simple @...

@eip-automerger
Copy link

eip-automerger commented Apr 10, 2018

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

  • EIP 908 has no identifiable authors who can approve PRs

@jamesray1
Copy link
Contributor Author

jamesray1 commented Apr 10, 2018

Suggestions?

@5chdn
Copy link
Contributor

5chdn commented Apr 10, 2018 •
edited
Loading

From what I understand:

Real Name (@ Github Name) <Email Address>

@Arachnid
Copy link
Contributor

Arachnid commented Apr 10, 2018

Use either Real Name (@githubid) or Real Name <email address>.

@nicksavers
Copy link
Contributor

nicksavers commented Apr 11, 2018

The delimiter should be a comma.

@Arachnid Arachnid merged commit bd06a2e into ethereum:master Apr 11, 2018
Arachnid pushed a commit to Arachnid/EIPs that referenced this pull request May 2, 2018
@jamesray1 @Arachnid
Add Github usernames (ethereum#990)
15b1350 
* Add Github usernames

* Remove about.me link from triangular brackets

* Add @ before usernames, also note the previous commit was adding content from the original PR.

ethereum#908

* parentheses instead of triangular brackets: (@githubusername)

* replace and with ,
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jamesray1 @5chdn @nicksavers @eip-automerger @Arachnid