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

EIPIP Meeting 108 #350

Closed
2 of 3 tasks
poojaranjan opened this issue Jul 19, 2024 · 6 comments
Closed
2 of 3 tasks

EIPIP Meeting 108 #350

poojaranjan opened this issue Jul 19, 2024 · 6 comments

Comments

@poojaranjan
Copy link
Member

poojaranjan commented Jul 19, 2024

Date and Time

Aug 21, 2024 at 14:00 UTC

Location

Zoom: TBA in the Discord #eip-editing channel

YouTube Recording: EIPIP Meetings

Agenda

1. Discuss Open Issues/PRs, and other topics

Edit to Final Proposals

Update to EIP-1

TBA

Misc Issue/ requires attention?

TBA

Call for Input

Call For Input Status Inclination/Result Deadline
343 Closed Merged July 28, 2024
345 Closed Delete Aug 02, 2024
348 Closed Merged Aug 17, 2024
349 Open Aug 16, 2024

2. Other discussions and updates from past meetings

TBA

3. EIPs Insight - Monthly EIPs status reporting.

Repo Issues PRs Total
EIP 3 50 53
ERC 6 71 77
RIP 3 14 17

4. EIP Editing Office Hour

  • Agenda 43 Video 42

5. Review action items from earlier meetings

Next Meeting date & time - Sep 18, 2024 (3rd Wed of the Month) at 14:00 UTC

@poojaranjan poojaranjan mentioned this issue Jul 20, 2024
10 tasks
@smartprogrammer93
Copy link

Hey team,
I found another instance of FORK_TIMESTAMP variable that is not specified for Dencun:
https://eips.ethereum.org/EIPS/eip-7045
Not sure if you want to act on it similar to 4788 or leave it. Thought that since there is precedence to fixing such issues, that we can fix them all.

@poojaranjan
Copy link
Member Author

@smartprogrammer93
As @SamWilsn mentioned we can take it case by case.
Most likely, a Call For Input will be created if we receive a Pull Request to update the EIP.
The final decision will be of the editors. However, I'd be in favor of merging the PR.

@SamWilsn
Copy link
Collaborator

Can we add "disabling the stale bot" to the agenda?

It's just annoying and doesn't take into account whether an issue/PR is waiting on author or editor.

@viwinkumarpadala
Copy link

viwinkumarpadala commented Aug 21, 2024

Hey team,
I want to provide a demo for our website: EIPsInsight
We have created tooling designed to provide clear, visual insights into the activity of Ethereum Improvement Proposal (EIP), Ethereum Request for Comments (ERCs), and Rollup Improvement Proposals (RIPs) over a specified period. Data provided can be used for tracking the progress all the proposals. We have also added a few tools that can be useful for editors for keeping track of PRs and Reviews.

@poojaranjan
Copy link
Member Author

Summary

1. Discuss Open Issues/PRs, and other topics

disabling the stale bot

  • @SamWilsn proposed to disable the bot as it is closing PRs before the review
  • @lightclient suggested to edit/update the bot instead of disabling it completely
  • Todos
    • @SamWilsn will create PRs to
    • Update the bot to continue providing warnings with edits to 6 months
    • Send a message to provide 1 week warning before closing
      The ideal expectation will be to have the bot close PR waiting for the author and not the editors

Ref: PR for EIP and [PR for ERC]ethereum/ERCs#600)

FORK_TIMESTAMP variable that is not specified for Dencun - https://eips.ethereum.org/EIPS/eip-7045

Edit to Final Proposals

ethereum/EIPs#8817

  • @SamWilsn created this PR to fix a small typo
  • Need 3 EIP Editors confirmation to merge the PR

Call For Input

#349

  • This is the deadline. There are only two comments, one in favor and one in against.
  • It would require more Editors comment to make a decision.

3. EIP Insight

  • @poojaranjan shared the demo for tooling added
    • Status Timeline: Pictorial representation of the change of status with number of days in each status
    • PRs & Issues demo shared.
      • Sam Wilson suggested that the chart can be updated to Bar or Line chart against the timeline.
    • Editors Review is breaking with date parameter. The team will take a look fix it.
      • The chart against timeline will be good to have.
  • Todo for EIPsInsight team

The next meeting is scheduled for Sep 18, 2024 at 14:00 UTC

@poojaranjan
Copy link
Member Author

Closing in favor #356

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

No branches or pull requests

4 participants