kitops icon indicating copy to clipboard operation
kitops copied to clipboard

[feat:] Added Config Command to Kitops CLI

Open SkySingh04 opened this issue 1 year ago • 2 comments

Description

This PR introduces the new config command for the KitOps CLI. It allows users to manage the configuration of CLI settings more easily. With this command, users can set, get, list, and reset configuration values for persistent options like storage paths, credentials, logging levels, and other preferences.

Key additions:

  • Config command: Introduced the config command with four subcommands:

    • set: Allows users to set configuration values.
    • get: Retrieves the value of a specified configuration key.
    • list: Lists all current configuration options and their values.
    • reset: Resets all configurations back to default values.
  • Persistent root flags: The root command now uses configuration settings from the config.json file for default values like --log-level, --progress, and --verbose, while still allowing overrides via command-line arguments.

Note: This PR supports Config for only root command flags, as it is not clearly mentioned if it should be implemented for all flags.

Linked Issues

Closes issue #419

SkySingh04 avatar Oct 09 '24 22:10 SkySingh04

@amisevsk Could you please review this PR and suggest upon how we can improve it?

SkySingh04 avatar Oct 09 '24 22:10 SkySingh04

@amisevsk Your requested changes have been made :D

SkySingh04 avatar Oct 11 '24 07:10 SkySingh04

@amisevsk Your requested changes have been made!

SkySingh04 avatar Oct 17 '24 00:10 SkySingh04

Can you add the license header for config.go?

gorkem avatar Oct 17 '24 13:10 gorkem

@gorkem license header has been added!

SkySingh04 avatar Oct 17 '24 18:10 SkySingh04

@amisevsk @gorkem Waiting for a response on this so that we can proceed further!

SkySingh04 avatar Oct 21 '24 14:10 SkySingh04

@gorkem Your requested changes have been made, could you kindly review?

SkySingh04 avatar Oct 24 '24 11:10 SkySingh04

@amisevsk Your requested changes have been made!

SkySingh04 avatar Nov 01 '24 16:11 SkySingh04

@gorkem @amisevsk Any reason for this PR to be marked as draft? Any changes that are still pending?

SkySingh04 avatar Dec 17 '24 18:12 SkySingh04