Skip to content

ci: only run the full test suite if FORM source code changes - #898

Merged
jodavies merged 1 commit into
form-dev:masterfrom
jodavies:ci-conditional
Sep 15, 2026
Merged

jodavies merged 1 commit into
form-dev:masterfrom
jodavies:ci-conditional

Conversation

@jodavies

Copy link
Copy Markdown
Collaborator

There is no need to run the full (time consuming) test suite if the source code has not changed (for eg, updating a README etc). The Deploy workflow at least compiles form and runs make check, so this would detect problems due to changes to the build configuration or workflow config files.

@jodavies
jodavies force-pushed the ci-conditional branch 2 times, most recently from 3efe6d8 to c3a5af1 Compare September 3, 2026 08:22
@coveralls

coveralls commented Sep 3, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 65.735%. remained the same — jodavies:ci-conditional into form-dev:master

@jodavies

jodavies commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

Updated: the full suite should also run if test.yml itself changes.

The full test suite takes a long time to run, particularly under valgrind.
Only run it if FORM's source code, the tests, or the yml file controlling
the tests are modified.

Other changes (just to the documentation, for eg) will only run the fast
"deploy" tests.
@jodavies
jodavies merged commit d145584 into form-dev:master Sep 15, 2026
119 of 120 checks passed
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