Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

update chain-id format #542

Merged
merged 15 commits into from
Sep 24, 2020
Merged

update chain-id format #542

merged 15 commits into from
Sep 24, 2020

Conversation

fedekunze
Copy link
Contributor

Closes: #540

Description


For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@fedekunze fedekunze requested a review from noot as a code owner September 24, 2020 17:03
@fedekunze fedekunze mentioned this pull request Sep 24, 2020
@fedekunze fedekunze changed the title update chai-id format update chain-id format Sep 24, 2020
@codecov
Copy link

codecov bot commented Sep 24, 2020

Codecov Report

Merging #542 into development will decrease coverage by 0.05%.
The diff coverage is 80.76%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #542      +/-   ##
===============================================
- Coverage        70.77%   70.72%   -0.06%     
===============================================
  Files               41       42       +1     
  Lines             2320     2333      +13     
===============================================
+ Hits              1642     1650       +8     
- Misses             536      540       +4     
- Partials           142      143       +1     
Impacted Files Coverage Δ
types/chain_id.go 61.53% <61.53%> (ø)
app/ante/eth.go 58.62% <100.00%> (ø)
x/evm/handler.go 91.37% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b9a10b3...e0e07b6. Read the comment docs.

@@ -57,6 +61,7 @@ Ref: https://keepachangelog.com/en/1.0.0/

* (ante) [\#525](https://github.com/ChainSafe/ethermint/pull/525) Add message validation decorator to `AnteHandler` for `MsgEthereumTx`.
* (types) [\#507](https://github.com/ChainSafe/ethermint/pull/507) Fix hardcoded `aphoton` on `EthAccount` balance getter and setter.
* (types) [\#501](https://github.com/ChainSafe/ethermint/pull/501) Fix bech32 encoding error by using the compressed ethereum secp256k1 public key.
Copy link
Contributor Author

@fedekunze fedekunze Sep 24, 2020

Choose a reason for hiding this comment

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

forgot to add this bug changelog entry

run: |
make test-sim-multi-seed-short
if: "env.GIT_DIFF != ''"
# install-runsim:
Copy link
Contributor Author

@fedekunze fedekunze Sep 24, 2020

Choose a reason for hiding this comment

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

commented them because they are currently broken anyway

@fedekunze fedekunze merged commit a924b20 into development Sep 24, 2020
@fedekunze fedekunze deleted the chain-id branch September 24, 2020 17:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow ChainId as string
2 participants