Skip to content

Run required docs check on every PR - #257

Merged
krowvin merged 1 commit into
mainfrom
fix/required-html-check
Aug 11, 2026
Merged

Run required docs check on every PR#257
krowvin merged 1 commit into
mainfrom
fix/required-html-check

Conversation

@krowvin

@krowvin krowvin commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • run the Docs Validation workflow for every pull request
  • retain the existing path filters for pushes to main

Why

The main - required checks ruleset requires the html job for every pull request. The workflow's pull-request path filter prevented that job from starting when a change did not touch documentation, Python source, project metadata, scripts, or CONTRIBUTING.md. GitHub then left the required html context in an indefinitely expected state and blocked otherwise valid pull requests.

Impact

Every pull request now reports the required html status. Scheduled behavior and path-filtered validation after pushes to main remain unchanged.

Validation

  • actionlint -shellcheck= .github/workflows/docs.yml
  • commit-time yamlfix and generated ownership checks
  • confirmed the unrestricted pull-request trigger matches the repository ruleset's unconditional html requirement

actionlint with ShellCheck enabled still reports the existing SC2129 style warning in the workflow summary step; the same warning is present on origin/main and is outside this fix.

@krowvin
krowvin marked this pull request as ready for review August 11, 2026 21:57
@krowvin
krowvin requested a review from Enovotny as a code owner August 11, 2026 21:57
@krowvin
krowvin merged commit daf9feb into main Aug 11, 2026
12 checks passed
@krowvin
krowvin deleted the fix/required-html-check branch August 11, 2026 21:57
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