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 spell coordination schedule to the post-goerli-world #22

Merged
merged 9 commits into from
May 6, 2024

Conversation

SidestreamColdMelon
Copy link
Contributor

@SidestreamColdMelon SidestreamColdMelon commented Feb 12, 2024

Closes #23

Since we plan to deprecate goerli testnet after February 21, 2024, we also need to update our coordination schedule and remove outdated information. This PR:

  • Removes goerli from the schedule (since it's being deprecated)
  • Removes old cadence proposal linked in current checklists (since it becomes irrelevant)
  • Introduces basic table format instead of a gantt chart or a list
  • Introduces one-row-per-role structure to better outline sequence of events for the outsiders
  • Splits crafters checklists into relevant sections (to match stages described in the schedule)
  • Updates currently used schedule
    • Crafting still starts on Friday
    • Most of the deadlines are moved to 16:00 UTC instead of 23:59 CET
    • New "sync" session is proposed in the end of the spell to streamline handover process

@SidestreamColdMelon SidestreamColdMelon marked this pull request as ready for review February 13, 2024 08:27
@SidestreamColdMelon
Copy link
Contributor Author

@amusingaxl as you're fine with the proposal to have "two reviews" – one for the code based on the sheet, and one to validate addressed review + the exec doc, I would rearrange the checklist items to account for this process change. For example checks relevant to exec doc can go to the bottom of the "Development Stage" or even moved into a separate new section between "Development" and "Deployment", eg "Pre-Deployment". This way, the reviewers wouldn't have to jump around the checklists and hold publishing them till the very end

Copy link
Contributor

@amusingaxl amusingaxl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@VoteWizard VoteWizard left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

Choose a reason for hiding this comment

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

lgtm :)

@SidestreamColdMelon SidestreamColdMelon merged commit 1aa58ee into master May 6, 2024
2 checks passed
@SidestreamColdMelon SidestreamColdMelon deleted the update-spell-schedule branch May 6, 2024 07:39
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.

Update Spell Schedule (remove goerli and improve mainnet)
4 participants