Skip to content

fix: preserve coverage failure diagnostics - #22

Closed
AlbertoCentonze wants to merge 1 commit into
fix/skip-empty-gas-reportfrom
fix/preserve-coverage-diagnostics
Closed

AlbertoCentonze wants to merge 1 commit into
fix/skip-empty-gas-reportfrom
fix/preserve-coverage-diagnostics

Conversation

@AlbertoCentonze

@AlbertoCentonze AlbertoCentonze commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Coverage failures currently hide Forge stdout inside a truncated Node Buffer. In aave-helpers #806, all 311 ordinary tests passed before coverage failed, but 36,520 bytes of its stdout were omitted, leaving the cause unknown.

Run coverage directly in a Bash step so complete stdout and stderr reach the job log and a nonzero status fails the step. Disable the formatter action's embedded coverage command to avoid running coverage twice. Diagnostics stay in GitHub-masked job logs rather than unmasked raw-log artifacts.

@AlbertoCentonze
AlbertoCentonze added this pull request to stack #18 September 14, 2026 15:00
@AlbertoCentonze
AlbertoCentonze removed this pull request from stack #18 September 14, 2026 15:06
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