Skip to content

feat(release): add logUnchangedProjects flag to version generator, true by default#27231

Merged
JamesHenry merged 2 commits into
masterfrom
release-version-log-unchanged-projects-option
Aug 7, 2024
Merged

feat(release): add logUnchangedProjects flag to version generator, true by default#27231
JamesHenry merged 2 commits into
masterfrom
release-version-log-unchanged-projects-option

Conversation

@JamesHenry

@JamesHenry JamesHenry commented Jul 31, 2024

Copy link
Copy Markdown
Contributor

Current Behavior

The version generator always emits logs for all projects. For large monorepos, especially when only a subset of the many projects are released together, this can feel noisy.

Expected Behavior

The version generator can be configured via a new logUnchangedProjects generator option to not emit any logs for projects with no changes. The current default behavior is unchanged, the flag is opt in (i.e. setting it to false).

Related Issue(s)

Fixes #

@JamesHenry JamesHenry requested review from a team as code owners July 31, 2024 12:58
@JamesHenry JamesHenry requested a review from AgentEnder July 31, 2024 12:58
@vercel

vercel Bot commented Jul 31, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Aug 7, 2024 2:04pm

@JamesHenry JamesHenry changed the title feat(release): add logUnchangedProjects flag to generator, true by default feat(release): add logUnchangedProjects flag to version generator, true by default Jul 31, 2024
@JamesHenry JamesHenry enabled auto-merge (squash) July 31, 2024 12:58
@nx-cloud

nx-cloud Bot commented Jul 31, 2024

Copy link
Copy Markdown
Contributor

)}`
);
this.logs.forEach((msg) => {
console.log(this.color.instance.bold(this.projectName) + ' ' + msg);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
console.log(this.color.instance.bold(this.projectName) + ' ' + msg);
console.log(`${this.color.instance.bold(this.projectName)} ${msg}`);

@JamesHenry JamesHenry merged commit 5551823 into master Aug 7, 2024
@JamesHenry JamesHenry deleted the release-version-log-unchanged-projects-option branch August 7, 2024 21:22
ZackDeRose pushed a commit to ZackDeRose/nx that referenced this pull request Aug 8, 2024
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants