Skip to content

Move gas snapshot check to merge CI - #1327

Open
babymakh2026-maker wants to merge 1 commit into
aave:mainfrom
babymakh2026-maker:agent/run-gas-report-on-merge
Open

Move gas snapshot check to merge CI#1327
babymakh2026-maker wants to merge 1 commit into
aave:mainfrom
babymakh2026-maker:agent/run-gas-report-on-merge

Conversation

@babymakh2026-maker

@babymakh2026-maker babymakh2026-maker commented Aug 4, 2026

Copy link
Copy Markdown

Summary

  • Run the existing Foundry gas report action in the merge workflow.
  • Keep PR gas reporting and comment generation while removing PR snapshot enforcement.
  • Enforce FORGE_SNAPSHOT_CHECK from the merge CI test step.

Fixes #403

Validation

  • ruby -e 'require "yaml"; ARGV.each { |f| YAML.load_file(f) }; puts "YAML OK"' .github/workflows/tests-pr.yml .github/workflows/tests-merge.yml
  • yarn prettier --check .github/workflows/tests-pr.yml .github/workflows/tests-merge.yml
  • npx --yes github-actionlint@1.7.12 .github/workflows/tests-pr.yml .github/workflows/tests-merge.yml
  • git diff --check

Workflow-only change; no Forge tests were run locally.

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.

Run gas report in ci/cd

1 participant