Skip to content

fix: skip CI tests when only docs are changed (#2335)#2338

Merged
reynaldichernando merged 1 commit intoHeyPuter:mainfrom
serilevanjalines:fix-skip-docs-tests
Jan 24, 2026
Merged

fix: skip CI tests when only docs are changed (#2335)#2338
reynaldichernando merged 1 commit intoHeyPuter:mainfrom
serilevanjalines:fix-skip-docs-tests

Conversation

@serilevanjalines
Copy link
Copy Markdown
Contributor

Description

Skip test workflows (backend tests, API tests, vitest) when only documentation files in src/docs/ are modified.

Changes

  • Added paths-ignore: ["src/docs/**"] to both push and pull_request triggers in .github/workflows/test.yml

Why This Change?

After moving puter.js docs to this repo, CI workflows run unnecessarily for docs-only changes, wasting resources and time.

Testing

  • Documentation-only changes will now skip all test jobs
  • Code changes continue to trigger full test suite as before

Fixes #2335

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 24, 2026

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Member

@reynaldichernando reynaldichernando left a comment

Choose a reason for hiding this comment

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

LGTM, but lets see what copilot has to say

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 optimizes CI resource usage by skipping test workflows when only documentation files are changed. After the recent migration of puter.js docs to this repository, test workflows were running unnecessarily for documentation-only changes.

Changes:

  • Added paths-ignore: ["src/docs/**"] to both push and pull_request triggers in the test workflow
  • Tests will now be skipped when only files in src/docs/ are modified

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@reynaldichernando
Copy link
Copy Markdown
Member

reynaldichernando commented Jan 24, 2026

in this PR its still running the github actions
although this is just due to how it works, plus since we are working on the non docs folder

it will only be ignored on subsequent docs PRs

(i encountered this in the past)

@reynaldichernando reynaldichernando merged commit 7045452 into HeyPuter:main Jan 24, 2026
10 checks passed
@serilevanjalines serilevanjalines deleted the fix-skip-docs-tests branch January 24, 2026 17:11
@serilevanjalines serilevanjalines restored the fix-skip-docs-tests branch January 24, 2026 17:12
@serilevanjalines serilevanjalines deleted the fix-skip-docs-tests branch January 24, 2026 17:14
@serilevanjalines
Copy link
Copy Markdown
Contributor Author

@reynaldichernando Thanks for the quick review and merge!
Yeah, that makes sense , the paths-ignore filter will only apply to future PRs since this one was modifying the workflow itself. Appreciate the explanation !

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.

Skip github actions test if working on src/docs

4 participants