Skip to content

Deleted reference to baklava and alfajores#2175

Merged
palango merged 2 commits into
mainfrom
martinvol-patch-2
Apr 29, 2026
Merged

Deleted reference to baklava and alfajores#2175
palango merged 2 commits into
mainfrom
martinvol-patch-2

Conversation

@martinvol
Copy link
Copy Markdown
Contributor

No description provided.

@martinvol martinvol requested a review from a team as a code owner April 22, 2026 15:23
@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 22, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
celo-docs 🟢 Ready View Preview Apr 22, 2026, 3:37 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@viral-sangani
Copy link
Copy Markdown
Collaborator

@claude can you review this?

@claude
Copy link
Copy Markdown

claude Bot commented Apr 29, 2026

Claude finished @viral-sangani's task in 1m 3s —— View job


PR Review

  • Gather context and read changed files
  • Analyze the diff
  • Provide review feedback

Summary

This PR updates contribute-to-celo/release-process/smart-contracts.mdx to reflect Celo's transition from L1 (with Baklava and Alfajores testnets) to L2 (using Celo Sepolia). The changes are well-motivated and accurate.

Changes Reviewed

1. Network option updates in bash code blocks

Four bash code blocks are updated to replace deprecated testnet names:

  • ${"baklava"|"alfajores"|"mainnet"}${"anvil"|"celo-sepolia"|"mainnet"}
  • ${"anvil"|"sepolia"|"mainnet"}${"anvil"|"celo-sepolia"|"mainnet"} ✅ (corrects sepoliacelo-sepolia for consistency)

These are correct — celo-sepolia is the proper network identifier for the Celo L2 testnet.

2. Promotion process table

The old two-step testnet flow (Baklava at T+2w → Alfajores at T+3w → Mainnet at T+4w) is condensed into a single testnet step (Celo Sepolia at T+2w → Mainnet at T+3w). This is logically consistent since there is now only one testnet in the pipeline. The existing T+5w final step is preserved. ✅

3. Communication guidelines

  • "Baklava and Alfajores testnets" correctly changed to "Celo Sepolia testnet" ✅
  • Removed the Baklava-specific posting deadline, which is appropriate since that testnet no longer exists ✅

Minor Observations

These are pre-existing issues not introduced by this PR, noted for awareness:

  • Line 222: "Register a Vaildator" (typo)
  • Line 230: "Get a vaildator slashed" (typo)
  • Line 426: "CGP status as EXCECUTED" (typo)

Verdict

The changes are correct, clean, and appropriately scoped. The PR accurately reflects the L1→L2 migration by removing outdated Baklava/Alfajores references and replacing them with Celo Sepolia. Already approved by @palango. Looks good to merge.

@palango palango merged commit d5c6913 into main Apr 29, 2026
6 checks passed
@palango palango deleted the martinvol-patch-2 branch April 29, 2026 10:53
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.

3 participants