Skip to content

Conversation

@AmadiMichael
Copy link
Member

A semgrep rule that checks that all OPCM V2 variables are prefixed with opcm except otherwise allowed. That way, it is going to be identified as an OPCM component by the VerifyOPCM script so that its contractsContainer address is accounted for when checking consistency among all OPCM components

@AmadiMichael AmadiMichael requested a review from a team as a code owner December 22, 2025 17:29
@codecov
Copy link

codecov bot commented Dec 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.42%. Comparing base (1c9d0fc) to head (f8198b9).
⚠️ Report is 6 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #18679      +/-   ##
===========================================
+ Coverage    73.44%   77.42%   +3.98%     
===========================================
  Files          189      134      -55     
  Lines        11244     7213    -4031     
===========================================
- Hits          8258     5585    -2673     
+ Misses        2840     1628    -1212     
+ Partials       146        0     -146     
Flag Coverage Δ
cannon-go-tests-64 ?
contracts-bedrock-tests 77.42% <ø> (+0.61%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 56 files with indirect coverage changes

🚀 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.

@opgitgovernance opgitgovernance added the S-stale Status: Will be closed unless there is activity label Jan 6, 2026
@opgitgovernance
Copy link
Contributor

This pr has been automatically marked as stale and will be closed in 5 days if no updates

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-stale Status: Will be closed unless there is activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants