Skip to content

[Backport / 10.4]: Fix description of --no-logo option - #631

Merged
ChrisonSimtian merged 2 commits into
Fallout-build:release/v10.4from
IT-VBFK:backport/no-logo-descr-rename
Aug 13, 2026
Merged

[Backport / 10.4]: Fix description of --no-logo option#631
ChrisonSimtian merged 2 commits into
Fallout-build:release/v10.4from
IT-VBFK:backport/no-logo-descr-rename

Conversation

@IT-VBFK

@IT-VBFK IT-VBFK commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Finally closes #553 for 10.4

ChrisonSimtian and others added 2 commits August 11, 2026 12:23
Cover the descriptions --help prints and .fallout/build.schema.json embeds.
Two fail: --no-logo still describes the NUKE logo.

Part of Fallout-build#553

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit 3a46818)
The parameter description and its doc comment still named NUKE after the
rebrand, so --help and the generated .fallout/build.schema.json told consumers
the flag hides the NUKE logo.

Regenerate the checked-in schema and update the four SchemaUtilitySpecs
snapshots plus the inline expectation in CompletionUtilitySpecs, all of which
embed the description.

Closes Fallout-build#553

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit c14512d)
@IT-VBFK
IT-VBFK requested a review from a team as a code owner August 11, 2026 10:26

@ChrisonSimtian ChrisonSimtian left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks for backporting, can you version bump please as this is the first fix for this release?

@ChrisonSimtian ChrisonSimtian added Can Merge These PRs are just waiting to be merged, nothing more to do and maintainer has approved channel/support Support lines: security/critical fixes only -> nuget.org opt-in + GH Packages (ADR-0004). labels Aug 13, 2026
@IT-VBFK

IT-VBFK commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

No
Unfortunately only on my phone this week.

@ChrisonSimtian

Copy link
Copy Markdown
Collaborator

No Unfortunately only on my phone this week.

all good, I just merge it and raise a dedicated bump for the next patch release :-)

@ChrisonSimtian
ChrisonSimtian merged commit 228ae16 into Fallout-build:release/v10.4 Aug 13, 2026
3 checks passed
@IT-VBFK
IT-VBFK deleted the backport/no-logo-descr-rename branch August 13, 2026 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Can Merge These PRs are just waiting to be merged, nothing more to do and maintainer has approved channel/support Support lines: security/critical fixes only -> nuget.org opt-in + GH Packages (ADR-0004).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants