Skip to content

ci: do not build solver for docs only PR#2419

Merged
triceo merged 1 commit into
TimefoldAI:mainfrom
TomCools:build/ignore-docs-only-pr-build
Jun 25, 2026
Merged

ci: do not build solver for docs only PR#2419
triceo merged 1 commit into
TimefoldAI:mainfrom
TomCools:build/ignore-docs-only-pr-build

Conversation

@TomCools

Copy link
Copy Markdown
Contributor

Currently, when doing a PR just for docs, it still builds the entire matrix of the solver.
This is highly ineffecient. This PR seeks to resolve that, by excluding PRs involving only changes in docs/

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts GitHub Actions pull request workflow triggers so that solver/quickstarts/secure CI workflows do not run when a PR only changes files under docs/, relying instead on the dedicated documentation preview workflow.

Changes:

  • Add docs/** to paths-ignore in the base PR workflow to avoid running the full solver CI matrix for docs-only PRs.
  • Add docs/** to paths-ignore in the secured PR workflow to avoid running secret-bearing jobs for docs-only PRs.
  • Add docs/** to paths-ignore in the quickstarts PR workflow to avoid running quickstarts CI for docs-only PRs.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/pull_request.yml Skips the solver CI matrix on PRs that only change docs/**.
.github/workflows/pull_request_secure.yml Skips secured pull_request_target jobs on docs-only PRs.
.github/workflows/pull_request_quickstarts.yml Skips quickstarts CI on docs-only PRs.

@triceo triceo changed the title chore: do not build solver for docs only PR ci: do not build solver for docs only PR Jun 25, 2026
@triceo triceo merged commit 6a7712f into TimefoldAI:main Jun 25, 2026
22 of 25 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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.

3 participants