-
Notifications
You must be signed in to change notification settings - Fork 489
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
Change diverged flags back to Geth's convention (POS-602) #451
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #451 +/- ##
===========================================
+ Coverage 56.30% 56.36% +0.06%
===========================================
Files 597 600 +3
Lines 69014 69037 +23
===========================================
+ Hits 38855 38912 +57
+ Misses 26812 26798 -14
+ Partials 3347 3327 -20
Continue to review full report at Codecov.
|
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
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. Notice that integration test failed. This is because the flags are changed, so bor couldn't correctly launch in CI. After this is merged, we will need to change start.sh in matic-cli here: https://github.com/maticnetwork/matic-cli/blob/v0.3.0-dev/src/setup/devnet/templates/docker/docker-bor-start.sh.njk
…rk#451) * changed name -> identity * changed no-snapshot -> snapchot=true/false * changed jsonrpc.corsdomain -> http.corsdomain * changed jsonrpc.vhosts -> http.vhosts * changed http/ws.modules to http/ws.api * updated readme * updated config_test * make docs * handelling string array flag, overwrite insted of append * added 'Default' to SliceStringFlag * added separate flags for corsdomain and vhosts for http, ws, graphql * modified tests
* changed name -> identity * changed no-snapshot -> snapchot=true/false * changed jsonrpc.corsdomain -> http.corsdomain * changed jsonrpc.vhosts -> http.vhosts * changed http/ws.modules to http/ws.api * updated readme * updated config_test * make docs * handelling string array flag, overwrite insted of append * added 'Default' to SliceStringFlag * added separate flags for corsdomain and vhosts for http, ws, graphql * modified tests
This PR does the following things:
Default
field to theSliceStringFlag
.corsdomain
andvhosts
forhttp, ws, and graphql
.POS-602