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

Only load default plugins when necessary #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

smergler
Copy link

General Information

This is for a fix to issue #3.

We only load the default plugins when necessary.

Testing

For testing I was able to run the config discussed in the issue and received the following result:
Screenshot 2024-07-14 at 12 04 59 AM

I had also tried to run it with the problematic argument and received an expected error.
Screenshot 2024-07-14 at 12 07 30 AM

Side note

Admittedly this is not best practice in terms of style, which would require having the imports at the top of the file. However, this is required due to the way that dotbot imports it's plugins.

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.

1 participant