Skip to content

Comments

chore: Enable mainnet#623

Merged
mergify[bot] merged 3 commits intosigp:release-v1.0.0from
dknopik:mainnet
Sep 26, 2025
Merged

chore: Enable mainnet#623
mergify[bot] merged 3 commits intosigp:release-v1.0.0from
dknopik:mainnet

Conversation

@dknopik
Copy link
Member

@dknopik dknopik commented Sep 19, 2025

Proposed Changes

  • Allow passing mainnet to network.
  • Remove mainnet safeguard from client startup function.
  • Adjust comments and docs.
  • Make mainnet the default network.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables mainnet support in the Anchor validator by removing testnet restrictions and making mainnet the default network configuration.

  • Removed mainnet safeguard from client startup that prevented mainnet usage
  • Updated default network from "hoodi" to "mainnet"
  • Added mainnet contract address and updated documentation to reflect mainnet support

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/docs/pages/running_an_operator.mdx Updated documentation to include mainnet as a supported network option
docs/docs/pages/cli.mdx Changed default network parameter from "hoodi" to "mainnet" in CLI documentation
anchor/keysplit/README.md Updated examples and documentation to use mainnet instead of hoodi, removed mainnet restriction note
anchor/eth/README.md Added mainnet contract address and updated description to include mainnet support
anchor/common/global_config/src/lib.rs Changed default hardcoded network from "hoodi" to "mainnet" and added mainnet to allowed values
anchor/client/src/lib.rs Removed mainnet genesis fork version check that blocked mainnet usage
Comments suppressed due to low confidence (1)

anchor/client/src/lib.rs:1

  • The constant MAINNET_GENESIS_FORK_VERSION is removed but not used elsewhere in the visible diff. Since this was used for the mainnet check that's being removed, this cleanup is appropriate, but ensure no other references to this constant exist in the codebase.
pub mod cli;

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dknopik dknopik added ready-for-review This PR is ready to be reviewed v1.0.0 First Mainnet-release and removed do-not-merge labels Sep 23, 2025
@dknopik dknopik added ready-for-merge and removed ready-for-review This PR is ready to be reviewed labels Sep 26, 2025
@dknopik
Copy link
Member Author

dknopik commented Sep 26, 2025

LFG! 🥳

@mergify mergify bot added the queued label Sep 26, 2025
mergify bot added a commit that referenced this pull request Sep 26, 2025
@mergify mergify bot merged commit 95121c6 into sigp:release-v1.0.0 Sep 26, 2025
18 checks passed
@mergify mergify bot removed the queued label Sep 26, 2025
diegomrsantos pushed a commit to diegomrsantos/anchor that referenced this pull request Oct 6, 2025
- Allow passing `mainnet` to `network`.
- Remove `mainnet` safeguard from client startup function.
- Adjust comments and docs.
- Make `mainnet` the default network.


Co-Authored-By: Daniel Knopik <daniel@dknopik.de>
jking-aus pushed a commit to jking-aus/anchor that referenced this pull request Oct 8, 2025
- Allow passing `mainnet` to `network`.
- Remove `mainnet` safeguard from client startup function.
- Adjust comments and docs.
- Make `mainnet` the default network.


Co-Authored-By: Daniel Knopik <daniel@dknopik.de>
petarjuki7 pushed a commit to petarjuki7/anchor that referenced this pull request Oct 16, 2025
- Allow passing `mainnet` to `network`.
- Remove `mainnet` safeguard from client startup function.
- Adjust comments and docs.
- Make `mainnet` the default network.


Co-Authored-By: Daniel Knopik <daniel@dknopik.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge v1.0.0 First Mainnet-release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants