Skip to content

Conversation

@manuelbarbas
Copy link
Contributor

The SKALE Base chain mainnet now has a new chain Id

This PR updates the Id:

  • Old Id: 1562508942
  • New Id: 1187947933

Copy link
Member

@onbjerg onbjerg left a comment

Choose a reason for hiding this comment

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

Hi, please follow these instructions to make CI pass:

chains/src/named.rs

Lines 8 to 26 in 442ebff

// When adding a new chain:
// 1. add new variant to the NamedChain enum;
// 2. add extra information in the last `impl` block (explorer URLs, block time) when applicable;
// 3. (optional) add aliases:
// - Strum (in kebab-case): `#[strum(to_string = "<main>", serialize = "<aliasX>", ...)]`
// `to_string = "<main>"` must be present and will be used in `Display`, `Serialize`
// and `FromStr`, while `serialize = "<aliasX>"` will be appended to `FromStr`.
// More info: <https://docs.rs/strum/latest/strum/additional_attributes/index.html#attributes-on-variants>
// - Serde (in snake_case): `#[cfg_attr(feature = "serde", serde(alias = "<aliasX>", ...))]`
// Aliases are appended to the `Deserialize` implementation.
// More info: <https://serde.rs/variant-attrs.html>
// - Add a test at the bottom of the file
// 4. run `cargo test --all-features` to update the JSON bindings and schema.
// 5. run `cargo +nightly fmt --all` to properly format the code.
// We don't derive Serialize because it is manually implemented using AsRef<str> and it would break
// a lot of things since Serialize is `kebab-case` vs Deserialize `snake_case`. This means that the
// NamedChain type is not "round-trippable", because the Serialize and Deserialize implementations
// do not use the same case style.

@zerosnacks zerosnacks enabled auto-merge (squash) December 11, 2025 15:48
@zerosnacks zerosnacks merged commit 6386b31 into alloy-rs:main Dec 11, 2025
20 checks passed
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