Skip to content

an attempt to handle empry matrix on release#1410

Open
jacobheric wants to merge 1 commit intomainfrom
jacob/release-ci
Open

an attempt to handle empry matrix on release#1410
jacobheric wants to merge 1 commit intomainfrom
jacob/release-ci

Conversation

@jacobheric
Copy link
Copy Markdown
Contributor

@jacobheric jacobheric commented Mar 25, 2026

Summary

Not sure what changed recently, but when a PR has no prerelease labels, the matrix filter produces [] and fail the job. This is an attempt to handle that.

Checklist

  • Added a docs PR that references this PR
  • Added unit/integration tests
  • Added changesets if applicable

Related

  • INN-

Note

Adds a has_packages output to the generate-prerelease-matrix job and gates the prereleases job behind an if condition that skips execution when no prerelease/* labels are present on the PR, preventing a matrix-with-empty-include failure.

Written by Mendral for commit 6b3163d.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 25, 2026

⚠️ No Changeset found

Latest commit: 6b3163d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Copy Markdown

@mendral-app mendral-app Bot left a comment

Choose a reason for hiding this comment

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

LGTM

The fix is correct. jq -n '$data | length > 0' emits the bare string true/false, and the if: ... == 'true' string comparison in GitHub Actions handles it properly. No issues.

Tag @mendral-app with feedback or questions. View session

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.

1 participant