Skip to content

Ignore doc changes for Codex review Action#1571

Merged
rolandwalker merged 1 commit intomainfrom
RW/ignore-doc-files-codex-action
Feb 17, 2026
Merged

Ignore doc changes for Codex review Action#1571
rolandwalker merged 1 commit intomainfrom
RW/ignore-doc-files-codex-action

Conversation

@rolandwalker
Copy link
Contributor

@rolandwalker rolandwalker commented Feb 16, 2026

Description

  • Skip Codex review when the changes are documentation-only.
  • Sync all workflows to the same paths-ignore spec.
  • Use a single **.rst property.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Feb 16, 2026
@rolandwalker rolandwalker changed the title Ignore doc changes for Codex review action Ignore doc changes for Codex review Action Feb 16, 2026
@github-actions
Copy link

Findings
No correctness, security, or test-gap issues found in the changes.

Optional
If the intent is to skip Codex for all documentation-only PRs including assets, consider adding screenshots/** or a broader doc/** pattern to paths-ignore so image-only doc updates don’t trigger the workflow. (.github/workflows/codex-review.yml)

@rolandwalker rolandwalker force-pushed the RW/ignore-doc-files-codex-action branch from 1cc88bf to 90f60d9 Compare February 16, 2026 11:43
@github-actions
Copy link

No findings.

I reviewed the changes in .github/workflows/codex-review.yml and changelog.md and didn’t spot correctness regressions or security concerns. The paths-ignore behavior is standard: the workflow will still run if any non-ignored file changes.

Test gaps: none. This is a workflow filter change. If you want extra assurance, you could add a minimal CI-only PR that edits only a doc file to confirm the workflow does not run, but it’s optional.

@rolandwalker rolandwalker force-pushed the RW/ignore-doc-files-codex-action branch from 90f60d9 to ab5f261 Compare February 16, 2026 22:18
paths-ignore:
- '**.md'
- 'LICENSE.txt'
- 'AUTHORS.rst'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a bunch of rst files. Can we do '**.rst' ?

 * skip Codex review when the changes are documentation-only
 * sync all workflows to the same paths-ignore spec
@rolandwalker rolandwalker force-pushed the RW/ignore-doc-files-codex-action branch from ab5f261 to 04fba9c Compare February 16, 2026 23:00
@rolandwalker rolandwalker requested a review from amjith February 16, 2026 23:01
@rolandwalker rolandwalker merged commit bf218fc into main Feb 17, 2026
8 checks passed
@rolandwalker rolandwalker deleted the RW/ignore-doc-files-codex-action branch February 17, 2026 08:49
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