Skip to content

Commit 080bafa

Browse files
ondrejmirtesclaude
andcommitted
Fix manual dispatch to check all parameters, not just diff
Use github.event.before for push diffs to handle multi-commit pushes. Add explicit else branch for manual dispatch to check entire schema. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 547f128 commit 080bafa

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

.github/workflows/document-config-params.lock.yml

Lines changed: 4 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/document-config-params.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,9 @@ Also skip these internal parameters that users should not configure directly:
6969
Check which parameter names from the schema do NOT appear as documented parameters in `config-reference.md`. A parameter counts as "documented" if it appears as a heading (`###`), in a config key listing, or is explained in a section body.
7070

7171
{{#if github.event_name == 'push'}}
72-
Focus on parameters that were added or changed in the push. Run `git diff HEAD~1 -- conf/parametersSchema.neon` to see what changed. Only document newly added parameters.
72+
Focus only on parameters that were added or changed in this push. Run `git diff ${{ github.event.before }} -- conf/parametersSchema.neon` to see what changed across all commits in the push. Only document newly added parameters.
73+
{{#else}}
74+
Check ALL non-skipped parameters from the schema against the documentation. Do not look at git history or diffs — compare the entire `parametersSchema.neon` against `config-reference.md` and document every undocumented parameter you find.
7375
{{/if}}
7476

7577
If there are no undocumented parameters, stop and report that all parameters are documented. Do not create a PR.

0 commit comments

Comments
 (0)