Add publish-plan command#2061
Conversation
🦋 Changeset detectedLatest commit: a0febcf The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2061 +/- ##
==========================================
+ Coverage 81.27% 81.75% +0.48%
==========================================
Files 62 64 +2
Lines 2350 2390 +40
Branches 645 648 +3
==========================================
+ Hits 1910 1954 +44
+ Misses 395 393 -2
+ Partials 45 43 -2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| } | ||
|
|
||
| if (packagesToPublish.length !== 0) { | ||
| log.info( |
There was a problem hiding this comment.
should stuff like this be logged when --output gets used?
There was a problem hiding this comment.
I think we should keep it for now so it's consistent with changeset status, but we can revisit this later if it's better to remove.
bluwy
left a comment
There was a problem hiding this comment.
LGTM. Just one small nit/preference to keep the output as internal.
This kinda addresses #614 (albeit in a different way)
closes #200