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

Meta SNIP-23 Procedure for Introducing Starknet Breaking Changes #102

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

nconsigny
Copy link
Contributor

No description provided.

@nconsigny nconsigny changed the title Meta snip Meta snip "Procedure for Introducing Starknet Breaking Changes" Jul 26, 2024
@nconsigny nconsigny changed the title Meta snip "Procedure for Introducing Starknet Breaking Changes" Draft meta snip "Procedure for Introducing Starknet Breaking Changes" Jul 26, 2024
SNIPS/snip-XXX.md Outdated Show resolved Hide resolved
SNIPS/snip-XXX.md Outdated Show resolved Hide resolved
SNIPS/snip-XXX.md Outdated Show resolved Hide resolved
SNIPS/snip-XXX.md Outdated Show resolved Hide resolved
@Eikix
Copy link
Collaborator

Eikix commented Jul 29, 2024

Is there an EIP number for the same process? If so, we can name this SNIP after the same number. Otherwise, I'll come back in a few days and attribute it the next available number.

@nconsigny nconsigny marked this pull request as ready for review August 1, 2024 13:24
SNIPS/snip-XXX.md Outdated Show resolved Hide resolved
@nconsigny nconsigny changed the title Draft meta snip "Procedure for Introducing Starknet Breaking Changes" Meta snip - Procedure for Introducing Starknet Breaking Changes Aug 18, 2024
@nconsigny nconsigny requested a review from Eikix August 18, 2024 16:53
@Eikix
Copy link
Collaborator

Eikix commented Aug 27, 2024

@iliav-starkware posting here what we discussed during our meeting a month ago:

  • Starkware will start making every major feature in the Starknet roadmap a SNIP for visibility
  • @Eikix to hold accountable for 0.13.3

Once this is approved we can merge the SNIP and count it approved

@Eikix
Copy link
Collaborator

Eikix commented Sep 23, 2024

Sync'd with @leo-starkware this morning about the above

@iliav-starkware posting here what we discussed during our meeting a month ago:

  • Starkware will start making every major feature in the Starknet roadmap a SNIP for visibility
  • @Eikix to hold accountable for 0.13.3

Once this is approved we can merge the SNIP and count it approved

SW is in theory in agreement to start writing SNIPs that incorporate all the added snips for a particular upgrade.

SNIPS/snip-XXX.md Outdated Show resolved Hide resolved
removed block target and updated SNIP number
@nconsigny nconsigny changed the title Meta snip - Procedure for Introducing Starknet Breaking Changes Meta SNIP-23 Procedure for Introducing Starknet Breaking Changes Oct 22, 2024
@l-henri
Copy link

l-henri commented Nov 6, 2024

Hello, any update for this SNIP?

@@ -0,0 +1,110 @@
---
snip: 23
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please change to 24. Discussion on numbering policy is ongoing here

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.

5 participants