Skip to content

feat(create-release-branch): group repeated warnings - #162

Draft
GuillaumeRx wants to merge 4 commits into
gr/crb-output-02-errorsfrom
gr/crb-output-03-warnings
Draft

GuillaumeRx wants to merge 4 commits into
gr/crb-output-02-errorsfrom
gr/crb-output-03-warnings

Conversation

@GuillaumeRx

@GuillaumeRx GuillaumeRx commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Explanation

Third step of the output revamp: repeated per-package warnings become one warning with a list of packages.

  • Functions now return what happened, and the core prints it once:
    • readProject returns packagesUsingRootTag. runRelease prints one warning for these packages before the front end starts.
    • The changelog functions return 'updated' | 'no-changelog' | 'no-updates'. Mode.prepare and Mode.apply turn these into Notice objects ({ title, items }).
  • startRelease and applyRelease print the notices after their step with a new CliOutput.warnList method.
  • The legacy mode, selectMode and determineInitialParameters no longer take output, because they no longer print.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

@GuillaumeRx
GuillaumeRx added this pull request to stack #161 November 5, 2026 17:14
@GuillaumeRx GuillaumeRx changed the title gr/crb output 03 warnings feat(create-release-branch): group repeated warnings Nov 5, 2026
@GuillaumeRx
GuillaumeRx force-pushed the gr/crb-output-03-warnings branch from 171b3db to c6f9918 Compare November 5, 2026 17:49
@GuillaumeRx
GuillaumeRx force-pushed the gr/crb-output-03-warnings branch from c6f9918 to 83218a0 Compare November 5, 2026 18:16

This branch has not been deployed

No deployments
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