Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Add block tag support for eth_feeHistory #4224

Merged
merged 5 commits into from
Aug 5, 2021
Merged

Add block tag support for eth_feeHistory #4224

merged 5 commits into from
Aug 5, 2021

Conversation

@spacesailor24
Copy link
Contributor

@spacesailor24 spacesailor24 commented Aug 5, 2021

No description provided.

brg8 and others added 2 commits August 4, 2021 21:53
@brg8 @spacesailor24
Accept predefined block numbers for eth_feeHistory (#4213)
9dde5a7 
* Accept predefined block numbers for eth_feeHistory

* Update feeHistory tests to reflect new input formatter

Co-authored-by: Wyatt Barnes <[email protected]>
@spacesailor24
Add block tag tests
b1b977c
@spacesailor24 spacesailor24 added the 1.x 1.0 related issues label Aug 5, 2021
@render
Copy link

render bot commented Aug 5, 2021

Your Render PR Server URL is https://web3-js-pr-4224.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c45l91sobjd906i8mae0.

@coveralls
Copy link

coveralls commented Aug 5, 2021 •
edited
Loading

Pull Request Test Coverage Report for Build 1100077324

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 254 unchanged lines in 7 files lost coverage.
  • Overall coverage increased (+2.4%) to 75.007%
Files with Coverage Reduction New Missed Lines %
packages/web3-core-requestmanager/src/jsonrpc.js 1 70.0%
packages/web3-core-helpers/src/formatters.js 6 84.39%
packages/web3-core-helpers/src/errors.js 29 1.56%
packages/web3-utils/src/utils.js 31 8.46%
packages/web3-utils/src/soliditySha3.js 34 3.43%
packages/web3-utils/src/index.js 43 31.38%
packages/web3-eth-accounts/src/index.js 110 26.12%
Totals Coverage Status
Change from base Build 1100070664: 2.4%
Covered Lines: 3202
Relevant Lines: 4033
💛 - Coveralls

@spacesailor24 spacesailor24 merged commit 29febb8 into 1.x Aug 5, 2021
@spacesailor24 spacesailor24 deleted the wyatt/eth_feeHistory-blocktags branch August 5, 2021 03:56
@spacesailor24 spacesailor24 mentioned this pull request Aug 5, 2021
Merged
@affiliatedkat affiliatedkat mentioned this pull request Feb 2, 2024
Open
@vutting4221 vutting4221 mentioned this pull request Feb 2, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@luu-alex luu-alex luu-alex approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@GregTheGreek GregTheGreek

@luu-alex luu-alex

@jdevcs jdevcs

Labels

1.x 1.0 related issues

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@spacesailor24 @coveralls @luu-alex @GregTheGreek @jdevcs @brg8