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

Update EIP-1: Adds @xinbenlv as an EIP Editor #5502

Merged
merged 3 commits into from
Sep 21, 2022

Conversation

Pandapip1
Copy link
Member

@Pandapip1 Pandapip1 requested a review from eth-bot as a code owner August 18, 2022 19:50
@eth-bot
Copy link
Collaborator

eth-bot commented Aug 18, 2022

A critical exception has occurred:
Message: pr 5502 is already merged; quitting
(cc @alita-moore, @mryalamanchi)

@xinbenlv
Copy link
Contributor

Thanks for the invitation @Pandapip1 .
I love to contribute to this group and am honored to be invited. I humbly accept the invitation if you and other editors also think my editorial contributions has been helpful than harmful.

Side note: your encouragement means a lot to me. I particularly admire this invitation from you because you and I usually hold very orthogonal views.

@xinbenlv
Copy link
Contributor

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

(fail) .github/workflows/auto-review-bot.yml

classification
ambiguous

  • '.github/workflows/auto-review-bot.yml' must be in eip-###.md format; this error will be overwritten upon relevant editor approval

It seems the eip-bot says it has to be eip-###.md format, is this intended? @Pandapip1

@poojaranjan
Copy link
Contributor

Having a good number of EIP Editors to review proposals to bring the number of open pull requests under 10 is a dream that I hope will become true one day.

This PR was brought up in EIPIP meeting 63, but we didn't get enough time to discuss. Though, this will be a decision of present EIP editors, I just want to share info documented in EIP editor “apprentice” handbook which is a collection of summary of decisions of past EIPIP meetings.

As per How can an apprentice make a request to be formally onboarded as an EIP editior?, it appears to me that "tenure of active contribution" is an important criteria of consideration along with approval of present EIP Editors.

"The contribution of an apprentice will be monitored by all EIP editors during the internship period. After 6 months of reviewing the pull requests as apprentice, when the contributor feels comfortable enough with the process and would like to be added to the repository as an EIP editor, a pull request with request to be listed as EIP editor has to be made."

IMHO, EIP Editor is a responsible position and building community trust will be helpful with contributions made over a period of time.

I see the mention of "internship duration" or "active contribution tenure" is missing in EIP-5069: EIP Editor Handbook. Probably it can be updated, if we consider engagement time as a valid consideration.

@Pandapip1
Copy link
Member Author

engagement time as a valid consideration

@xinbenlv has been engaged for at least the past month. That's close to (if not quite) how long I was engaged before I became an editor.

@Pandapip1 Pandapip1 added e-review Waiting on editor to review and removed waiting: editor to review labels Aug 26, 2022
@Pandapip1 Pandapip1 changed the title Adds @xinbenlv as an EIP Editor Update EIP-1: Adds @xinbenlv as an EIP Editor Aug 28, 2022
@xinbenlv
Copy link
Contributor

Thanks for the support @Pandapip1.

If it helps, my reviews can be found as: https://github.com/ethereum/EIPs/pulls?q=+reviewed-by%3AXinbenlv

@Pandapip1
Copy link
Member Author

If it helps, my reviews can be found as: https://github.com/ethereum/EIPs/pulls?q=+reviewed-by%3AXinbenlv

That will definitely be helpful!

@lightclient
Copy link
Member

I would personally like to see a longer history of consistent contributions, before adding as an editor.

@Pandapip1
Copy link
Member Author

I would personally like to see a longer history of consistent contributions, before adding as an editor.

Fair enough. I'll leave this open for now. @xinbenlv, just keep doing what you're doing for now.

@Pandapip1 Pandapip1 added e-consensus Waiting on editor consensus and removed e-review Waiting on editor to review labels Sep 15, 2022
@eth-bot eth-bot enabled auto-merge (squash) September 21, 2022 14:13
@eth-bot eth-bot merged commit 26cbfd2 into ethereum:master Sep 21, 2022
@SamWilsn
Copy link
Contributor

Greg and myself were on today's EIPIP call, and were in favour. Panda previously expressed a positive sentiment. I believe we now have sufficient evidence that @xinbenlv will be a solid contributor.

SamWilsn added a commit to SamWilsn/EIPs that referenced this pull request Sep 21, 2022
SamWilsn added a commit that referenced this pull request Sep 21, 2022
SamWilsn pushed a commit to SamWilsn/EIPs that referenced this pull request Sep 21, 2022
@Pandapip1 Pandapip1 deleted the patch-12 branch December 2, 2022 21:29
nachomazzara pushed a commit to nachomazzara/EIPs that referenced this pull request Jan 13, 2023
nachomazzara pushed a commit to nachomazzara/EIPs that referenced this pull request Jan 13, 2023
eth-bot pushed a commit that referenced this pull request Feb 8, 2023
* Adds @xinbenlv as an EIP Editor (#5502)

* Add xinbenlv to new config file

---------

Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>
iseriohn pushed a commit to iseriohn/EIP-NFT-Rights-Management that referenced this pull request Feb 16, 2023
* Adds @xinbenlv as an EIP Editor (ethereum#5502)

* Add xinbenlv to new config file

---------

Co-authored-by: Pandapip1 <45835846+Pandapip1@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e-consensus Waiting on editor consensus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants