Chore: Update chain tomls #844
Conversation
|
I see you updated files related to |
| [HeadTracker] | ||
| # Polygon suffers from a tremendous number of re-orgs, we need to set this to something very large to be conservative enough | ||
| HistoryDepth = 2000 | ||
| BlockHistorySize = 12 |
There was a problem hiding this comment.
current mainnet config assigned to NOPS uses finalityTag
There was a problem hiding this comment.
Could we please also include Wemix, Kroma & Gnosis in the same PR? I can help you add it, if we are okay to include it here
stackman27
force-pushed
the
sish/new-tomls-1
branch
from
929752a
to
a035f21
Compare
| @@ -0,0 +1,19 @@ | |||
| ChainID = '59902' | |||
| ChainType = 'metis' | |||
| FinalityDepth = 1 | |||
There was a problem hiding this comment.
Can you please add a comment here saying, that this will need to be changed
|
|
||
| [HeadTracker] | ||
| HistoryDepth = 2000 | ||
| BlockHistorySize = 12 |
There was a problem hiding this comment.
I think this should be a lot more for CCIP, I believe this is copied over from DF, same for Celo
| [HeadTracker] | ||
| # Polygon suffers from a tremendous number of re-orgs, we need to set this to something very large to be conservative enough | ||
| HistoryDepth = 2000 | ||
| BlockHistorySize = 12 |
There was a problem hiding this comment.
Could we please also include Wemix, Kroma & Gnosis in the same PR? I can help you add it, if we are okay to include it here
Motivation
Solution