Skip to content

Exclude text non-doc files from triggering full CI runs#64584

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:optimize-selective-checks-text-files
Apr 1, 2026
Merged

Exclude text non-doc files from triggering full CI runs#64584
potiuk merged 1 commit intoapache:mainfrom
potiuk:optimize-selective-checks-text-files

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented Apr 1, 2026

Changes to .txt and .md files that are not part of the doc build should not
trigger the full test suite. This is an optimization for CI runs that are triggered
by text, non-doc-build changes (e.g. editing INTHEWILD.md, README.md, or
.txt files).

Changes:

  • Add a new TEXT_NON_DOC_FILES file group matching *.txt and *.md files
  • Add only_text_non_doc_files_changed property to SelectiveChecks
  • Skip full test runs on PUSH/SCHEDULE/WORKFLOW_DISPATCH events when only text
    non-doc files changed
  • Skip canary runs when only text non-doc files changed
  • Subtract text non-doc files from the "remaining files" set so they don't
    trigger tests through the catch-all path
  • Remove the overly broad ^dev/.* pattern from the test-always file group
    since dev/ changes should be evaluated by more specific file group patterns

Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Claude Opus 4.6)

Generated-by: Claude Code (Claude Opus 4.6) following the guidelines

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
@potiuk
Copy link
Copy Markdown
Member Author

potiuk commented Apr 1, 2026

Transient / UI issues only.

@potiuk potiuk merged commit aca2d59 into apache:main Apr 1, 2026
143 of 147 checks passed
@potiuk potiuk deleted the optimize-selective-checks-text-files branch April 1, 2026 18:42
github-actions bot pushed a commit that referenced this pull request Apr 1, 2026
…uns (#64584)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 1, 2026

Backport successfully created: v3-2-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-2-test PR Link

github-actions bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Apr 1, 2026
…uns (apache#64584)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Apr 3, 2026
…64584)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
potiuk added a commit to potiuk/airflow that referenced this pull request Apr 6, 2026
…uns (apache#64584)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
potiuk added a commit that referenced this pull request Apr 6, 2026
…uns (#64584) (#64809)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)
vatsrahul1001 pushed a commit that referenced this pull request Apr 8, 2026
…uns (#64584) (#64809)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)
vatsrahul1001 pushed a commit that referenced this pull request Apr 15, 2026
…uns (#64584) (#64809)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)
vatsrahul1001 pushed a commit that referenced this pull request Apr 15, 2026
…uns (#64584) (#64809)

Changes to .txt and .md files that are not part of the doc build
should not trigger the full test suite. This adds a new
TEXT_NON_DOC_FILES file group to selective checks and an
only_text_non_doc_files_changed property so these files are
subtracted from the "remaining files" set, avoiding unnecessary
CI runs for text-only, non-doc-build changes.

Also removes the overly broad `^dev/.*` pattern from the
test-always file group since dev/ changes should be evaluated
by more specific file group patterns.
(cherry picked from commit aca2d59)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-2-test Mark PR with this label to backport to v3-2-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants