You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
The PR updates the hardhat devDependency from ^2.28.0 to ^2.28.5 in package.json. The TrieProof.test.js test suite is refactored to replace external Anvil process spawning with a fixture-based in-process approach. The new fixture deploys $TrieProof, StorageSlotMock, and CallReceiverMock contracts. Test setup now uses batchInBlock with plain transaction generator arrays, direct provider calls for eth_getProof operations, and BlockTries instance conversion for block data. Cryptographic expectations are updated to use '0x' string representations instead of ZeroBytes constants.
Possibly related PRs
Add test helper for Trie Proofs #6207: Modifies TrieProof.test.js to use BlockTries helper for manual trie and proof handling in the same test file.
Add TrieProof library #5826: Relates to TrieProof test suite modifications where this PR refactors the test structure to use fixtures while the related PR introduces the TrieProof library.
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.
Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check
❓ Inconclusive
The description only contains a PR checklist without substantive explanation of the changes or their purpose.
Add a meaningful description explaining why Hardhat was updated and how the eth_getProof integration improves the test infrastructure.
✅ Passed checks (1 passed)
Check name
Status
Explanation
Title check
✅ Passed
The title accurately summarizes both main changes: updating Hardhat to 2.8.5 and switching to eth_getProof from EDR, which match the changeset modifications.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing touches
📝 Generate docstrings
🧪 Generate unit tests (beta)
Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch update/hardhat-2.8.5
No actionable comments were generated in the recent review. 🎉
Comment @coderabbitai help to get the list of available commands and usage tips.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Checklist
npx changeset add)