Skip to content

Add review message on pre exit - #2270

Merged
bluwy merged 3 commits into
mainfrom
pre-exit-review
Aug 26, 2026
Merged

Add review message on pre exit#2270
bluwy merged 3 commits into
mainfrom
pre-exit-review

Conversation

@bluwy

@bluwy bluwy commented Aug 25, 2026

Copy link
Copy Markdown
Member

closes #240

Add a simple message on pre exit to ask to review the changesets in .changeset/pre.

NOTE: I updated the string formatting in the second commit, as the review message (added in the first commit) made the formatting a bit awkward. But I'm happy to revert this if we feel it's unnecessary for now.

@netlify

netlify Bot commented Aug 25, 2026

Copy link
Copy Markdown

Deploy Preview for changesets ready!

Name Link
🔨 Latest commit d56afd6
🔍 Latest deploy log https://app.netlify.com/projects/changesets/deploys/6a8d8dcc7c1aaf0008c4e173
😎 Deploy Preview https://deploy-preview-2270--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.

@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.56%. Comparing base (2eb65ba) to head (d56afd6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2270   +/-   ##
=======================================
  Coverage   84.56%   84.56%           
=======================================
  Files          80       80           
  Lines        3174     3174           
  Branches      895      895           
=======================================
  Hits         2684     2684           
  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 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.

it's a good guide for new users and reminder for old ones

@changeset-bot

changeset-bot Bot commented Aug 25, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d56afd6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@changesets/cli Patch

Not sure what this means? Click here to learn what changesets are.

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

@bluwy
bluwy added this pull request to the merge queue Aug 26, 2026
Merged via the queue into main with commit a0a4dc6 Aug 26, 2026
18 checks passed
@bluwy
bluwy deleted the pre-exit-review branch August 26, 2026 09:45
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.

Exiting pre mode should prompt for changeset cleanup

3 participants