Skip to content

Always print the resolved config before verifying in config file mode - #38

Merged
jcp19 merged 1 commit into
mainfrom
claude/printconfig-before-gobra-d9sp88
Aug 10, 2026
Merged

Always print the resolved config before verifying in config file mode#38
jcp19 merged 1 commit into
mainfrom
claude/printconfig-before-gobra-d9sp88

Conversation

@jcp19

@jcp19 jcp19 commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@jcp19
jcp19 force-pushed the claude/printconfig-before-gobra-d9sp88 branch from 5ea7b18 to 337194c Compare August 10, 2026 07:34
@jcp19
jcp19 marked this pull request as ready for review August 10, 2026 07:39
Instead of exposing `printConfig` as an input of the Action, run Gobra
once with `--printConfig` whenever `configFile` is set, so that the
options a run actually uses are visible in the log of every config file
mode run. `--printConfig` makes Gobra exit without verifying anything,
so it needs its own run, which happens before the verification time is
measured. The step fails without verifying if that run fails.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Kf6tkneAFHgZ5zaUYvw4JN
@jcp19
jcp19 force-pushed the claude/printconfig-before-gobra-d9sp88 branch from 337194c to c49e8c4 Compare August 10, 2026 07:41
@jcp19
jcp19 merged commit 2278f6b into main Aug 10, 2026
4 checks passed
@jcp19
jcp19 deleted the claude/printconfig-before-gobra-d9sp88 branch August 10, 2026 07:52
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.

2 participants