Skip to content

Record compiler warning output in dice example - #6

Merged
DerekCorniello merged 3 commits into
mainfrom
feat/compiler-warning-baselines
Aug 28, 2026
Merged

Record compiler warning output in dice example#6
DerekCorniello merged 3 commits into
mainfrom
feat/compiler-warning-baselines

Conversation

@DerekCorniello

Copy link
Copy Markdown
Contributor

Updates the golden output for dice_simulation to include the new W0300 unused-binding diagnostics emitted by the paired compiler.

This keeps the shared examples runner deterministic while the compiler warning catalog is introduced.

This PR is intentionally ready for review (not a draft).

@greptile-apps

greptile-apps Bot commented Aug 28, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the dice simulation golden output to include unused-binding compiler warnings and adds support for selecting a paired compiler branch through a PR label.

  • Records the two W0300 diagnostics in the expected output.
  • Resolves and validates an optional paired-compiler:<branch> label before checking out the compiler.
  • Continues to use the compiler’s main branch when no paired branch is selected.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
.github/workflows/examples.yml Adds validated PR-label-based compiler ref selection while retaining main as the default.
examples/dice_simulation/expected_output.txt Updates the exact-output fixture with the compiler’s two W0300 unused-binding diagnostics.

Reviews (3): Last reviewed commit: "Document paired compiler validation" | Re-trigger Greptile

@DerekCorniello
DerekCorniello merged commit 1331e99 into main Aug 28, 2026
2 checks passed
@DerekCorniello
DerekCorniello deleted the feat/compiler-warning-baselines branch August 28, 2026 02:30
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