Skip to content

fix: skip empty gas report comments - #21

Merged
AlbertoCentonze merged 1 commit into
mainfrom
fix/skip-empty-gas-report
Sep 24, 2026
Merged

AlbertoCentonze merged 1 commit into
mainfrom
fix/skip-empty-gas-report

Conversation

@AlbertoCentonze

@AlbertoCentonze AlbertoCentonze commented Sep 2, 2026 •

Copy link
Copy Markdown
Contributor

Repositories without Forge gas snapshots receive a setup prompt on every pull request, as seen on aave-helpers #793.

Skip creating gas.txt when the upstream action returns its canonical “not measuring gas” placeholder. Real gas snapshot reports remain unchanged.

@AlbertoCentonze
AlbertoCentonze force-pushed the fix/skip-empty-gas-report branch from 1687a1c to 0fe8b81 Compare September 2, 2026 10:22
@AlbertoCentonze
AlbertoCentonze removed this pull request from stack #18 September 14, 2026 15:06
@AlbertoCentonze
AlbertoCentonze added this pull request to stack #23 September 14, 2026 15:06
@AlbertoCentonze
AlbertoCentonze removed this pull request from stack #23 September 24, 2026 08:34
@AlbertoCentonze
AlbertoCentonze force-pushed the fix/skip-empty-gas-report branch from 0fe8b81 to 7106fb9 Compare September 24, 2026 10:29
@AlbertoCentonze
AlbertoCentonze changed the base branch from chore/remove-zksync-ci to main September 24, 2026 10:29
@AlbertoCentonze
AlbertoCentonze merged commit aa7958c into main Sep 24, 2026
@AlbertoCentonze
AlbertoCentonze deleted the fix/skip-empty-gas-report branch September 24, 2026 10:38
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.

1 participant