Skip to content

Improve GHA workflows #267

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Improve GHA workflows #267

wants to merge 5 commits into from

Conversation

cwillisf
Copy link
Contributor

@cwillisf cwillisf commented Jul 18, 2025

Proposed Changes

Various improvements to the GHA workflows, especially for the build-and-test workflow.

  • Consider .nvmrc when caching node_modules, so we don't (for example) load Node 20 modules when we switch to Node 22
  • Move commitlint to its own separate workflow, so it doesn't block the build-and-test workflow
  • Move tests into a separate job matrix so it's easier to tell what fails and re-run them if necessary
  • Move path filters out of ci.yml (where the YAML must be embedded in a string) and into a separate YAML file (where they can be actual YAML and your editor can provide proper syntax highlighting)

At some point in the future, it'd be nice to matrix out the builds as well, but we'll need to eliminate cases where the build output of one package is an input to another.

Test Coverage

Runs existing tests

@cwillisf cwillisf requested a review from a team as a code owner July 18, 2025 15:19
@KManolov3
Copy link
Contributor

This looks great!

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