Skip to content

V4: Split test, build, and lint CI steps into their own jobs - #2988

Merged
CollinBeczak merged 1 commit into
maproulette:v4from
maxwellward:ci/split-build-test-and-lint
Nov 12, 2026
Merged

CollinBeczak merged 1 commit into
maproulette:v4from
maxwellward:ci/split-build-test-and-lint

Conversation

@maxwellward

Copy link
Copy Markdown

Splitting test, build, and lint into their own jobs improves readability in the CI pipeline. With this change you can see which individual step failed at a glance, rather than having to dig into the actions page and sift through the logs.

They can also run in parallel, speeding up CI runs a bit. This technically uses more runner minutes because npm ci is run on all 3 jobs, but won't make any real difference to hitting any free limits (there aren't even any limits for public projects).

Because all 3 run in parallel now, they don't block each other. If lint fails, test and build still get checked.

@jake-low jake-low 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.

Seems like a good idea to me!

@CollinBeczak
CollinBeczak merged commit 8e9f26d into maproulette:v4 Nov 12, 2026
4 checks passed
@maxwellward
maxwellward deleted the ci/split-build-test-and-lint branch November 12, 2026 02:56
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