Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

modify evm listener server #82

Merged
merged 17 commits into from
Dec 22, 2023
Merged

modify evm listener server #82

merged 17 commits into from
Dec 22, 2023

Conversation

robertcc0410
Copy link

Closes: #45

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)

outpoint := wire.NewOutPoint(inTxid, unspentTx.Vout)
txIn := wire.NewTxIn(outpoint, nil, nil)
tx.AddTxIn(txIn)
totalInputAmount += btcutil.Amount(unspentTx.Amount * 1e8)

Check notice

Code scanning / CodeQL

Floating point arithmetic Note

Floating point arithmetic operations are not associative and a possible source of non-determinism
@robertcc0410 robertcc0410 marked this pull request as ready for review December 22, 2023 06:33
@oxf71 oxf71 merged commit 18655e6 into b2network:dev Dec 22, 2023
25 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants