ci: only run tests for non-docs changes#9194
Conversation
c44902a to
d08aae8
Compare
|
I would reject this, as it opens a trap for a fatal error: The idea itself is good, but only valid for the following paths:
|
+1 |
|
oh in addition, I just realised how this works, we would want to have a new test target “build website” and run that if there are changes under We can skip all tests for changes in the .md/text files as outlined here. |
|
In theory I support this, but it may prevent use of github's "required workflows" feature, which blocks PRs from being merged if tests have not passed. |
Overview
This PR changes the ci workflow file in a way that it only runs if there are changes in paths that as not considered doc changes:
This only mirrors the path list that was already defined in the docs.yml. But could be extended with for example the CONTRIBUTING.md, TESTING.md and README.md.
here is the documentation regarding the
paths-ignoreoption in Github Workflows:https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#example-excluding-paths
Checklist
docsfolder