Skip to content

Split CI job and validate changesets#71

Merged
unflxw merged 1 commit into
mainfrom
split-ci-validate-changesets
Apr 2, 2026
Merged

Split CI job and validate changesets#71
unflxw merged 1 commit into
mainfrom
split-ci-validate-changesets

Conversation

@unflxw

@unflxw unflxw commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Split the single "ci" job into jobs with specific names, to make it easier to see what validation failed at a glance. Add changeset validation with mono changeset validate.

See appsignal/appsignal-wrap#36 and appsignal/appsignal-wrap#38 for equivalent changes on Wrap, used as reference here.

@unflxw
unflxw requested review from lipskis and tombruijn March 27, 2026 14:47
@unflxw unflxw self-assigned this Mar 27, 2026
@unflxw unflxw added the chore A small task that takes a day or two at the most. label Mar 27, 2026
Comment thread .github/workflows/publish_release.yaml

@tombruijn tombruijn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

One suggestion, otherwise looks good 👍

Split the single "ci" job into jobs with specific names, to make it
easier to see what validation failed at a glance. Add changeset
validation with `mono changeset validate` as a reusable workflow.
@unflxw
unflxw force-pushed the split-ci-validate-changesets branch from 0d924f5 to 104572b Compare April 2, 2026 15:47
@unflxw
unflxw merged commit 4139176 into main Apr 2, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore A small task that takes a day or two at the most.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants