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

Add Ronin network #3219

Merged
merged 4 commits into from
Jan 28, 2025
Merged

Add Ronin network #3219

merged 4 commits into from
Jan 28, 2025

Conversation

quybeans
Copy link
Contributor

Ronin is an EVM blockchain designed specifically for game developers. The Ronin blockchain and ecosystem is under actively development by Sky Mavis, the team behind Axie Infinity.

We aware that Ronin's chain ID 2020 has collision with Public Mint. Public Mint is undergoing scaling back and maintenance on a recent Discord announcement.

As Ronin is onboarding more dApps to its ecosystem, we are looking to settle this collision with Public Mint team and this repo maintainer.

Copy link
Member

@ligi ligi left a comment

Choose a reason for hiding this comment

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

their chain is still active - so I will not merge this unless they make a public (not some discord) announcement that they abandon this chain

@ligi ligi closed this Jul 31, 2023
@glitch-txs
Copy link

glitch-txs commented Dec 9, 2023

Hi @ligi ! Could we consider reopening this PR? Public Mint has been inactive for several months now, while Ronin has been growing and active.
They have also been very responsive to the Ethereum protocols, the main wallet supports EIP-6963 and CAIP-25. They are also registered in Viem's repo and WalletConnect explorer chain registry.
We noticed there were a chain mismatch due to this conflict as we use this chain registry for the EIP6963 website. Thanks!

@ligi
Copy link
Member

ligi commented Dec 9, 2023

the other chain is still producing blocks

@glitch-txs
Copy link

Hey @ligi could you reopen? Seems like the blockchain stopped working 6 months ago

@dwi
Copy link

dwi commented Aug 6, 2024

Hello @ligi, the Public Mint network is not producing blocks for 8 months now. Last tweet was 17 months ago. It seems to me that the chain is now abandoned and no longer active.

@ligi ligi reopened this Aug 22, 2024
@ligi
Copy link
Member

ligi commented Aug 22, 2024

@zaidhub @rafaeltorres77 can you please chime in here?
@dwi this PR will then for sure need the redFlag reusedChainId

@justinabsentia
Copy link

Bump

@quybeans
Copy link
Contributor Author

quybeans commented Sep 5, 2024

Red flag is added.

Copy link

This PR has no activity in a while - it will be closed soon.

@github-actions github-actions bot added the Stale label Oct 20, 2024
@glitch-txs
Copy link

Bump

@github-actions github-actions bot removed the Stale label Oct 21, 2024
@YaroShkvorets
Copy link
Contributor

Bump

@glitch-txs
Copy link

can we merge? @ligi 👍

@antazoey
Copy link

chainlist.org has updated to show Ronin instead of Public Mint. This should merge.

@dwi
Copy link

dwi commented Jan 22, 2025

@ligi Can we please merge this?

@viko42
Copy link

viko42 commented Jan 28, 2025

Why is that not validated yet?

@Rias0232022
Copy link

Ok

@maxshedra
Copy link

Can this be merged?

@ligi
Copy link
Member

ligi commented Jan 28, 2025

CI is red: so no merge
the other chain does not seem to be a blocker anymore - but the CI is

@Rias0232022
Copy link

Ok

@alexx855
Copy link

@ligi i see all green, what is red exactly?

@ligi
Copy link
Member

ligi commented Jan 28, 2025

the red CI is fixed now - but the PR needs redFlags reusedChainId

"chainId": 2020,
"networkId": 2020,
"slip44": 60,
"redFlags": ["reusedChainId"],
Copy link

Choose a reason for hiding this comment

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

@ligi it has the red flag

@ligi ligi merged commit 1e9b79a into ethereum-lists:master Jan 28, 2025
5 checks passed
Copy link

PR merged - please consider contributing some funds to lists.eth

@ligi
Copy link
Member

ligi commented Jan 28, 2025

ah right - it is there - should not do it between other things - but otherwise the notification might be drowned in 1000 others - merged

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.