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.

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

fix/non-kapitan-files-on-init-list: refactor populated files listing #918

Merged
merged 2 commits into from
Jan 31, 2023

Conversation

jkrzemin
Copy link
Contributor

@jkrzemin jkrzemin commented Jan 4, 2023

Former implementation showed all files in target directory and skipped some of generated files. Now only files generated are shown and formatting has been changed to more tree [1] like - from [2] to [3]

Closes #306

[1] https://linux.die.net/man/1/tree

[2]

Populated /tmp/fake with:
/tmp/fake
/tmp/fake/templates
/tmp/fake/templates/docs
         my_readme.md
/tmp/fake/components
/tmp/fake/components/other_component
         __init__.py

[3]

Populated /tmp/fake with:
inventory
    classes
        common.yml
        my_component.yml
    targets
        my_target.yml
templates
    scripts
        my_script.sh
    docs
        my_readme.md
components
    my_component
        my_component.jsonnet
    other_component
        __init__.py

fixes ae0b444

Former implementation showed all files in target directory and skipped
some of generated files. Now only files generated are shown and
formatting has been changed to more tree [1] like - from [2] to [3]

Closes kapicorp#306

[1] https://linux.die.net/man/1/tree

[2]
Populated /tmp/fake with:
/tmp/fake
/tmp/fake/templates
/tmp/fake/templates/docs
         my_readme.md
/tmp/fake/components
/tmp/fake/components/other_component
         __init__.py

[3]
Populated /tmp/fake with:
inventory
    classes
        common.yml
        my_component.yml
    targets
        my_target.yml
templates
    scripts
        my_script.sh
    docs
        my_readme.md
components
    my_component
        my_component.jsonnet
    other_component
        __init__.py

fixes ae0b444
@jkrzemin jkrzemin force-pushed the fix/non-kapitan-files-on-init-list branch from 7ac7442 to 5622bf3 Compare January 14, 2023 18:50
Copy link
Member

@ramaro ramaro left a comment

Choose a reason for hiding this comment

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

Thanks!

@ramaro ramaro merged commit e039d0f into kapicorp:master Jan 31, 2023
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.

A couple minor bugs with reporting in kapitan init
2 participants