Skip to content

Conversation

@azf20
Copy link
Contributor

@azf20 azf20 commented Nov 2, 2022

  • split mainnet and testnet into dedicated files
  • separate "how to" information from configuration
  • retain networks.md with pointers to the new files
  • add feature support matrix

Benefits:

  • separate instructions from configuration
  • prepare for addition of L2 with its own config

@azf20 azf20 requested review from Jannis and fordN November 2, 2022 12:29
@azf20 azf20 marked this pull request as ready for review November 25, 2022 00:01
@azf20
Copy link
Contributor Author

azf20 commented Nov 25, 2022

This is now ready for review, given the Council approval of the feature matrix cc @fordN @Jannis

@azf20 azf20 changed the title Update networks.md configuration [do not merge] Update networks.md configuration & add initial feature matrix Nov 25, 2022
Copy link
Contributor

@fordN fordN left a comment

Choose a reason for hiding this comment

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

Looking good! Just a small typo to fix before merging. Could you also squash or cleanup commit history before rebase & merge to keep the overall history meaningful?

@azf20 azf20 force-pushed the azf20/networks-configuration branch from 7ac0037 to de62c75 Compare November 29, 2022 21:31
@azf20
Copy link
Contributor Author

azf20 commented Nov 29, 2022

Done, thanks so much @fordN!

@azf20 azf20 merged commit 9fb0239 into main Nov 30, 2022
@azf20 azf20 deleted the azf20/networks-configuration branch November 30, 2022 09:45
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.

5 participants