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

Fixed plugin issue #2945

Closed
wants to merge 1 commit into from
Closed

Conversation

cschuchardt88
Copy link
Member

@cschuchardt88 cschuchardt88 commented Nov 6, 2023

closes #2943

@cschuchardt88
Copy link
Member Author

Sorry for the formatting. My analyzers did it.... 😵‍💫

@gsmachado
Copy link
Contributor

Hey @cschuchardt88
Yes, that's definitely a good addition!

I already had this change in my shelve here in VSCode. So, when I saw your PR I asked myself: should I comment on this PR, or should I simply push my changes as a PR? 😅

To save a bit your time, I decided to create a new one: #2946

I'd suggest sticking with my PR because I just changed 2 lines of code and I used System.AppContext.BaseDirectory.

@cschuchardt88
Copy link
Member Author

Ok, i like your solution better, I'll just close this one.

@cschuchardt88 cschuchardt88 deleted the Issue-2943 branch November 6, 2023 23:06
@cschuchardt88 cschuchardt88 restored the Issue-2943 branch November 11, 2023 17:29
@cschuchardt88 cschuchardt88 deleted the Issue-2943 branch November 11, 2023 17:30
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.

Plugins Class PluginsDirectory ArgumentNullException
2 participants