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

feat(args): Change config arguments to be a list #110

Closed
wants to merge 1 commit into from

Conversation

mbrsamsung
Copy link
Contributor

@mbrsamsung mbrsamsung commented Dec 10, 2024

Change config parse argument from a single path
to a list of paths.

Add a new base Resource class that handles multiple paths relative to the single path the Resource class takes as input.

This closes #106

@mbrsamsung mbrsamsung force-pushed the mbr/add-multiple-configs branch from 4b6444c to dd4dd5b Compare December 10, 2024 09:54
Change config parse argument from a single path
to a list of paths.

Add a new base Resource class that handles multiple paths
relative to the single path the Resource class takes as input.
@mbrsamsung mbrsamsung force-pushed the mbr/add-multiple-configs branch from dd4dd5b to fa7bf25 Compare December 10, 2024 09:58
@mbrsamsung mbrsamsung requested a review from safl December 10, 2024 10:06
@safl
Copy link
Collaborator

safl commented Dec 30, 2024

Before we pursue this further then we need to improve testing, e.g. the testrunner would stop working with these changes, however, there are no test to capture that. So, closing this for now.

@safl safl closed this Dec 30, 2024
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.

Multiple configuration-files
2 participants