Skip to content

Skip release PR in PR status workflow example - #2240

Merged
bluwy merged 3 commits into
changesets:mainfrom
marcalexiei:docs/pr-status-skip-release-pr
Aug 12, 2026
Merged

Skip release PR in PR status workflow example#2240
bluwy merged 3 commits into
changesets:mainfrom
marcalexiei:docs/pr-status-skip-release-pr

Conversation

@marcalexiei

@marcalexiei marcalexiei commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

The example comments on every pull request, including the release PR where changeset version has consumed all changesets, so the comment always reports that none is present.

This adds an if condition to the pr-status job skipping changeset-release/*
branches, matching what the Changesets GitHub Bot does internally.

@changeset-bot

changeset-bot Bot commented Aug 11, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 419b360

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

@netlify

netlify Bot commented Aug 11, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for changesets ready!

Name Link
🔨 Latest commit 419b360
🔍 Latest deploy log https://app.netlify.com/projects/changesets/deploys/6a7bcb51216ba30008b0a911
😎 Deploy Preview https://deploy-preview-2240--changesets.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Andarist
Andarist requested a review from bluwy August 11, 2026 18:43
@codecov

codecov Bot commented Aug 11, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.53%. Comparing base (76dad93) to head (419b360).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #2240   +/-   ##
=======================================
  Coverage   84.53%   84.53%           
=======================================
  Files          80       80           
  Lines        3168     3168           
  Branches      892      892           
=======================================
  Hits         2678     2678           
  Misses        433      433           
  Partials       57       57

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

beeequeue
beeequeue approved these changes Aug 11, 2026
View reviewed changes

@beeequeue beeequeue 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.

very nice improvement!

beeequeue
beeequeue reviewed Aug 11, 2026
View reviewed changes

@beeequeue beeequeue 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.

should we also add this to the pr-comment step?

Comment thread site/guide/_snippets/automating-non-blocking.yaml Outdated
bluwy
bluwy reviewed Aug 12, 2026
View reviewed changes
Comment thread site/guide/automating.md Outdated
Comment thread site/guide/_snippets/automating-non-blocking.yaml Outdated
@bluwy
Apply suggestions from code review
419b360 
Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>
@bluwy
bluwy enabled auto-merge August 12, 2026 01:25
@bluwy
bluwy added this pull request to the merge queue Aug 12, 2026
Merged via the queue into changesets:main with commit 02ac61a Aug 12, 2026
18 checks passed
@marcalexiei
marcalexiei deleted the docs/pr-status-skip-release-pr branch August 12, 2026 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@beeequeue beeequeue beeequeue approved these changes
@bluwy bluwy bluwy approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@marcalexiei @beeequeue @bluwy