Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🌱 Filesystem and config persistence cleanup #1980

Closed
wants to merge 5 commits into from
Closed

🌱 Filesystem and config persistence cleanup #1980

wants to merge 5 commits into from

Commits on Feb 4, 2021

  1. Inject filesystem from cli instead of creating several per command 

    Signed-off-by: Adrian Orive <[email protected]>
    @Adirio
    Adirio committed Feb 4, 2021
    Configuration menu
    Copy the full SHA
    7032eb3 View commit details
    Browse the repository at this point in the history

  2. Move boilerplate loading to the scaffolders to use the injected file … 

    …system

Signed-off-by: Adrian Orive <[email protected]>
    @Adirio
    Adirio committed Feb 4, 2021
    Configuration menu
    Copy the full SHA
    3bd3f23 View commit details
    Browse the repository at this point in the history

  3. Use the cli filesystem to persist the project config 

    Use PreRunE and PostRunE functions to handle configuration file loading and saving

Signed-off-by: Adrian Orive <[email protected]>
    @Adirio
    Adirio committed Feb 4, 2021
    Configuration menu
    Copy the full SHA
    cb79436 View commit details
    Browse the repository at this point in the history

  4. Replace the Subcommand.UpdateContext method for an explicit input-out… 

    …put Subcommand.UpdateMetadata

Signed-off-by: Adrian Orive <[email protected]>
    @Adirio
    Adirio committed Feb 4, 2021
    Configuration menu
    Copy the full SHA
    b5cd372 View commit details
    Browse the repository at this point in the history

  5. Store abstraction for persisitng Config 

    Signed-off-by: Adrian Orive <[email protected]>
    @Adirio
    Adirio committed Feb 4, 2021
    Configuration menu
    Copy the full SHA
    0266db0 View commit details
    Browse the repository at this point in the history