Skip to content

fix(cli): add Ubuntu and RCD to supported OS list in help text#489

Merged
arodage merged 1 commit intomainfrom
fix/update-cli-help-supported-os
Mar 23, 2026
Merged

fix(cli): add Ubuntu and RCD to supported OS list in help text#489
arodage merged 1 commit intomainfrom
fix/update-cli-help-supported-os

Conversation

@arodage
Copy link
Copy Markdown
Contributor

@arodage arodage commented Mar 23, 2026

Merge Checklist

  • The changes in the PR have been built and tested
  • Documentation has been updated to reflect the changes (or no doc update needed)
  • Ready to merge

Description

The root command --help text listed only 3 supported operating systems (EMT, Azure Linux, Wind River eLxr), but the tool actually supports 5. This PR adds the missing Ubuntu and RCD (Red Hat Compatible Distro) entries to match the providers registered in build.go.

Any Newly Introduced Dependencies

None.

How Has This Been Tested?

  • Verified with go build — compiles cleanly
  • Confirmed ./build/os-image-composer --help now shows all 5 supported OS providers

@arodage arodage requested a review from a team as a code owner March 23, 2026 18:05
Copilot AI review requested due to automatic review settings March 23, 2026 18:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the root CLI --help text to accurately reflect the OS providers that are already implemented/registered in the build command, ensuring user-facing documentation matches actual support.

Changes:

  • Add Ubuntu to the supported OS list shown in the root command help text.
  • Add RCD (Red Hat Compatible Distro) to the supported OS list shown in the root command help text.

Comment thread cmd/os-image-composer/main.go
Copy link
Copy Markdown
Contributor

@magerstam magerstam left a comment

Choose a reason for hiding this comment

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

LGTM

@arodage arodage merged commit f455f98 into main Mar 23, 2026
36 of 38 checks passed
@arodage arodage deleted the fix/update-cli-help-supported-os branch March 23, 2026 20:27
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.

4 participants