-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Conversation
5chdn
commented
Jan 2, 2019
•
edited
Loading
edited
- update hardcoded headers for Kovan #9918465
- update hardcoded headers for Ropsten #4743169
- update hardcoded headers for Ethereum #6993921
- ensure consistent formatting across the chain specs
- use tabs
- remove unnecessary spaces
- use hex over dec values
- rename Kovan to "Kovan Testnet"
- rename Ropsten to "Ropsten Testnet"
- rename Foundation to "Ethereum"
- move Ropsten from "test" to "ropsten" subdir
Light
Full
Waiting for full sync to finish ... |
Would we mind to keep the space after
we have
I do think that's a more common styling convention (for example, jq by default formats json this way) and our other json files are also formatted as such. |
}, | ||
"0x000000000000000000000000000000000000000a": { | ||
"balance": "0x0" | ||
}, |
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.
What's the reason we specify zero balance accounts? Does this even modify the state?
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.
The address is counted as "exists" from beginning (so it does modify the state).
I was using a JSON formatter. Restored the whitespaces, easier to review now. |
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.
👍 tested on Ropsten, Kovan and Mainnet
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
(Ah sorry I was reading diff in reverse!) |