Skip to content

ci: skip heavy test workflows for GenerateDocs and changelog changes#4056

Merged
andygrove merged 1 commit intoapache:mainfrom
andygrove:ci/skip-generate-docs-and-changelog
Apr 24, 2026
Merged

ci: skip heavy test workflows for GenerateDocs and changelog changes#4056
andygrove merged 1 commit intoapache:mainfrom
andygrove:ci/skip-generate-docs-and-changelog

Conversation

@andygrove
Copy link
Copy Markdown
Member

Which issue does this PR close?

Closes #.

Rationale for this change

spark/src/main/scala/org/apache/comet/GenerateDocs.scala only builds documentation, and dev/changelog/*.md files are release notes. Changes to either cannot affect runtime behavior, so they should not trigger the full Linux, macOS, Miri, Spark SQL, or Iceberg Spark SQL test matrices. Today, edits to GenerateDocs.scala kick off every heavy workflow.

What changes are included in this PR?

Adds spark/src/main/scala/org/apache/comet/GenerateDocs.scala and dev/changelog/*.md to the paths-ignore block on both push and pull_request triggers for:

  • .github/workflows/iceberg_spark_test.yml
  • .github/workflows/miri.yml
  • .github/workflows/pr_build_linux.yml
  • .github/workflows/pr_build_macos.yml
  • .github/workflows/spark_sql_test.yml

dev/changelog/*.md is already matched by the existing **.md entry, but listing it explicitly documents the intent.

How are these changes tested?

CI on this PR exercises the updated filters. The workflows above should be skipped when only GenerateDocs.scala or a changelog file changes.

Add spark/src/main/scala/org/apache/comet/GenerateDocs.scala and
dev/changelog/*.md to paths-ignore so edits limited to doc generation
source or release changelog entries do not trigger the full build and
Spark SQL test suites.
@andygrove andygrove merged commit c0fd8ec into apache:main Apr 24, 2026
134 checks passed
@andygrove andygrove deleted the ci/skip-generate-docs-and-changelog branch April 24, 2026 00:18
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