Skip to content

Conversation

randy-cro
Copy link

Related change in go-ethereum: crypto-org-chain/go-ethereum#21

Closes: #XXX

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)

@randy-cro randy-cro requested a review from a team as a code owner August 15, 2025 04:06
@randy-cro randy-cro requested review from JayT106 and songgaoye and removed request for a team August 15, 2025 04:06
@randy-cro randy-cro force-pushed the refactor/remove-active-precompiles-from-vm-context branch from 4aef8e7 to 0446cf9 Compare August 15, 2025 04:07
Copy link

codecov bot commented Aug 15, 2025

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 46.07%. Comparing base (3a866eb) to head (9e6b138).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
x/evm/types/tracer.go 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #691   +/-   ##
========================================
  Coverage    46.07%   46.07%           
========================================
  Files          186      186           
  Lines        18646    18646           
========================================
  Hits          8592     8592           
  Misses        9314     9314           
  Partials       740      740           
Files with missing lines Coverage Δ
x/evm/keeper/keeper.go 83.45% <100.00%> (ø)
x/evm/keeper/state_transition.go 72.29% <100.00%> (ø)
x/evm/types/tracer.go 35.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

socket-security bot commented Aug 15, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedgolang/​github.com/​crypto-org-chain/​go-ethereum@​v1.10.20-0.20250812022812-0af6c9899f11 ⏵ v1.10.20-0.20250815065500-a4fbafcae0dd7410010010060

View full report

@thomas-nguy thomas-nguy changed the title Refactor/remove active precompiles from vm context chore: remove active precompiles from vm context Aug 15, 2025
@randy-cro randy-cro force-pushed the refactor/remove-active-precompiles-from-vm-context branch from 5f57a29 to 9e6b138 Compare August 15, 2025 07:07
@randy-cro randy-cro merged commit b94064f into develop Aug 15, 2025
38 checks passed
@randy-cro randy-cro deleted the refactor/remove-active-precompiles-from-vm-context branch August 15, 2025 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants