-
Notifications
You must be signed in to change notification settings - Fork 316
docs: remove dead blockchain chains from contract addresses #3263
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
Conversation
Remove the following defunct chains from the documentation: EVM chains removed: - Canto (mainnet + testnet) - EOS EVM (mainnet + testnet) - EVMOS (mainnet + testnet) - Fantom (mainnet + testnet) - Horizen EON (mainnet) + GOBI (testnet) - Injective inEVM (mainnet + testnet) - ZKFair (mainnet + testnet) Solana/SVM chains removed: - Mantis Testnet - Mantis Mainnet CosmWasm chains removed: - Sei Atlantic 2 (Stable + Beta) - Sei Pacific 1 (Stable) Co-Authored-By: nidhi@dourolabs.xyz <nidhi2894@gmail.com>
|
Prompt hidden (unlisted session) |
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
5 Skipped Deployments
|
|
do you see any other chain that is no longer used ? |
|
I don't see any obviously dead chains beyond the ones you explicitly requested to remove. However, I did notice some potentially outdated testnets that use deprecated networks:
These are based on Goerli/Mumbai which are being deprecated in favor of Sepolia/Amoy, but I didn't remove them since Pyth may still have active deployments there that some developers rely on. Let me know if you'd like me to remove any of these in this PR or a follow-up, or if you'd like a more data-driven audit (e.g., checking which chains have had no price updates recently). |
nidhi-singh02
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
do the data drive approach to check. |
Data-Driven Analysis ResultsI checked each of the potentially outdated testnets by visiting their block explorers. Here are the findings:
Recommendation: All 5 testnets should be removed from the documentation. The explorers are either completely down or show no activity, confirming these networks are deprecated. Would you like me to remove these in this PR or create a follow-up PR? |
create a new PR to remove these. |
Summary
Remove defunct blockchain chains from the Pyth price feed contract addresses documentation. These chains are no longer active and should not be listed in the docs.
EVM chains removed:
Solana/SVM chains removed:
CosmWasm chains removed:
Rationale
These blockchain networks are no longer operational or have discontinued their Pyth integration. Keeping them in the documentation creates confusion for developers and misrepresents the current state of Pyth's supported networks.
How has this been tested?
Documentation-only changes to MDX files. Verified that the correct chains were removed and that active chains (e.g., "Injective EVM") were preserved.
Reviewer Checklist
Link to Devin run: https://app.devin.ai/sessions/1a58796a8d36456f88b8cdfeee02dd4a
Requested by: nidhi@dourolabs.xyz (@nidhi-singh02)