Skip to content

chore: remove mainnet and sepolia deploy configs - #423

Merged
jackchuma merged 3 commits into
mainfrom
chore/remove-mainnet-sepolia-deploy-configs
Aug 27, 2026
Merged

chore: remove mainnet and sepolia deploy configs#423
jackchuma merged 3 commits into
mainfrom
chore/remove-mainnet-sepolia-deploy-configs

Conversation

@jackchuma

@jackchuma jackchuma commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

What changed? Why?

  • Removed the checked-in deploy-config/mainnet.json and deploy-config/sepolia.json files so deployment-specific values are no longer maintained in the repository.
  • Added an ignore rule for local deploy configs and updated the multiproof NoNitro guide to copy deploy-config/local.json to deploy-config/sepolia.local.json.
  • Documented the required non-zero Base Sepolia l2BlockTime and l2GenesisTimestamp fields, and updated the guide commands to use the ignored local config.

Notes to reviewers

The local deployment config remains available as the starting schema; users must populate the current Sepolia values in their ignored copy before deploying.

How has it been tested?

  • git diff --check
  • git check-ignore -v deploy-config/sepolia.local.json
  • Verified the documented Base Sepolia L2 timing values against the removed Sepolia config.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cb-heimdall

cb-heimdall commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

jackchuma and others added 2 commits August 27, 2026 09:42
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@jackchuma
jackchuma merged commit 703c657 into main Aug 27, 2026
6 of 7 checks passed
@jackchuma
jackchuma deleted the chore/remove-mainnet-sepolia-deploy-configs branch August 27, 2026 14:07
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