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

Make config.yaml and values.yaml optional #100

Closed

sl4vr wants to merge 4 commits into master from optional_yamls

Contributor

sl4vr commented May 20, 2021 •

edited

Loading

Make config.yaml and values.yaml optional
Check for lookup path presence instead of root path when dir_ok is True
Refactor find_files function

Fixes #70

sl4vr added 2 commits

May 20, 2021 21:26


          fix(project): make config.yaml and values.yaml optional

46a0237


          test(cli): add tests for optional yamls

1bb6838

sl4vr requested review from AFriemann and a team

May 20, 2021 18:46

sl4vr added 2 commits

May 21, 2021 00:11


          fix(project): check for lookup path presence instead of root path


          refactor(project): refactor find_files function

728cdcc

Contributor Author

sl4vr commented Nov 18, 2022

Closing stale PR

sl4vr closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet