Skip to content

Removed config show command#41

Merged
arodage merged 1 commit intomainfrom
remove-config-show-command
Jun 9, 2025
Merged

Removed config show command#41
arodage merged 1 commit intomainfrom
remove-config-show-command

Conversation

@arodage
Copy link
Copy Markdown
Contributor

@arodage arodage commented Jun 9, 2025

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • [] Ready to merge

Description

Removed the image-composer config show subcommand to simplify the CLI interface.

Remove config show command

Summary

Removes the image-composer config show subcommand to simplify the CLI interface.

Impact

  • image-composer config init - Still works
  • image-composer config show - No longer available
  • Users can manually view config files instead of using the show command

Any Newly Introduced Dependencies

How Has This Been Tested?

  • Verified config init functionality remains intact
  • Confirmed config show returns "unknown command" error
  • Updated help text reflects only available subcommands

@arodage arodage merged commit e350d98 into main Jun 9, 2025
1 check passed
magerstam pushed a commit that referenced this pull request Jun 10, 2025
@arodage arodage changed the title Removed config how command Removed config show command Jun 10, 2025
arodage added a commit that referenced this pull request Jun 10, 2025
arodage added a commit that referenced this pull request Jun 10, 2025
* image test

* updated the imagedisc to use azurelinux properly

* Updated imagedisc and test files

* updated go.mod

* removed logs

* fixed bytes -> MiB conversion

* added convert pkg and moved PartitionInfo

* Update README.md

* Renaming the local variable 'logger' to 'log'

This is to avoid shadowing the imported package name,
improve the code clarity.

Signed-off-by: Yu, Fei <fei.yu@intel.com>

* removed config how command (#41)

* restructured based on new repo layout

* fixed incorrect map lookup

---------

Signed-off-by: Yu, Fei <fei.yu@intel.com>
Co-authored-by: arodage <alpesh.ramesh.rodage@intel.com>
Co-authored-by: Yu, Fei <fei.yu@intel.com>
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.

2 participants