forked from ethereum/EIPs
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Update EIP-1: Add option to use GitHub username and email at once (et…
…hereum#6543) * Update EIP-1: Add option to use GitHub username and email at once * Fix some things that I missed
- Loading branch information
1 parent
fc7e593
commit addf612
Showing
1 changed file
with
6 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -154,13 +154,15 @@ or | |
|
||
> Random J. User (@username) | ||
if the email address or GitHub username is included, and | ||
or | ||
|
||
> Random J. User | ||
> Random J. User (@username) <[email protected]> | ||
if the email address is not given. | ||
if the email address and/or GitHub username is included, and | ||
|
||
> Random J. User | ||
It is not possible to use both an email and a GitHub username at the same time. If important to include both, one could include their name twice, once with the GitHub username, and once with the email. | ||
if neither the email address nor the GitHub username are given. | ||
|
||
At least one author must use a GitHub username, in order to get notified on change requests and have the capability to approve or reject them. | ||
|
||
|