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

Separate testnets and mainnets in aggregated chain list(s) #449

Open
geoknee opened this issue Aug 8, 2024 · 1 comment
Open

Separate testnets and mainnets in aggregated chain list(s) #449

geoknee opened this issue Aug 8, 2024 · 1 comment
Labels
M-good-first-issue Meta: Good for newcomers

Comments

@geoknee
Copy link
Collaborator

geoknee commented Aug 8, 2024

https://docs.google.com/document/d/1lbNN_p8wuk3EhC1erlI3P3-dh7lx4UzWxA5Egf2Xugo/edit#heading=h.reolvji23szx

Blocked on a decision about exactly how we should split them out.

For example:

  1. same file, but all mainnets come first and then all testnets
  2. same file, ordered as (1) but nested under a "testnets"/"mainnets" key
  3. a separate file for each list
@tessr
Copy link
Contributor

tessr commented Aug 9, 2024

I propose Option 3: Split chainlist into two separate files, one for mainnets and one for testnets. I think consumers of these lists would want to just see mainnets, or just testnets. We can have mainnets.toml/mainnets.json and testnets.toml/testnets.json.

@geoknee geoknee added the M-good-first-issue Meta: Good for newcomers label Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-good-first-issue Meta: Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants