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

1/COSS: Proposal Add Github Usernames #644

Closed
wants to merge 3 commits into from
Closed

Conversation

jimstir
Copy link
Contributor

@jimstir jimstir commented Dec 12, 2023

Proposal to add github usernames for editors and contributors along with emails.

@@ -5,7 +5,7 @@ name: (Waku v2 | Logos | Codex) RFC Template
status: (raw|draft|stable)
category: (Standards Track|Informational|Best Current Practice)
tags: an optional list of tags, not standard
editor: Daniel Kaiser <danielkaiser@status.im>
editor: Daniel Kaiser <https://github.com/kaiserd>
Copy link

@felicio felicio Dec 12, 2023

Choose a reason for hiding this comment

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

OP: #641 (comment)

and what I'd propose, for example

Suggested change
editor: Daniel Kaiser <https://github.com/kaiserd>
editor: John Doe <johndoe@status.im> johndoe

as

name [<email>] [gh handle]

kinda like comment on a ssh key instead of drooping the email altogether

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@felicio sorry, changed to correct proposal

Copy link

Choose a reason for hiding this comment

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

@jimstir you don't have to apologize 😅, it's just something I've just suggested 🙂.

Copy link

Choose a reason for hiding this comment

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

others might disagree, of course

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@felicio you can add other relevant reviewers

Copy link

Choose a reason for hiding this comment

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

@jimstir I can't, I don't have the rights on this repo 🙂

@jimstir
Copy link
Contributor Author

jimstir commented Mar 1, 2024

Continue discussion: vacp2p/rfc-index#4. The RFC process has been changed separating raw specs and the draft/stable specs into different repositories.

@jimstir jimstir closed this Mar 1, 2024
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