Skip to content

Conversation

@pedmiston
Copy link

Augments the Config class to accept an optional filename parameter to
specify a custom config file. Adds an argument to the CLI to pass the config
file through to the Config class.

Adds tests to ensure existing functionality is maintained, and paths are
expanded to abspaths properly. Adds a class variable "persist" to the Config
class to prevent writing files to disk during testing.

Augments the Config class to accept an optional filename parameter to
specify a custom config file. Adds an argument to the CLI to pass the config
file through to the Config class.

Adds tests to ensure existing functionality is maintained, and paths are
expanded to abspaths properly.  Adds a class variable "persist" to the Config
class to prevent writing files to disk during testing.
@pedmiston
Copy link
Author

👋 I'm a user of wpm and appreciate all the work that has gone into it! I'd like to offer to contribute to the project. I believe this MR would resolve #42 and may allow us to close #43 as a compromise solution. This would resolve a desire of mine to be able to specify a custom config file as well.

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.

2 participants