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

Fix plugins still needing specific python version. #315

Closed
wants to merge 1 commit into from
Closed

Fix plugins still needing specific python version. #315

wants to merge 1 commit into from

Conversation

mhertz
Copy link
Contributor

@mhertz mhertz commented Oct 27, 2021

No description provided.

@cas--
Copy link
Member

cas-- commented Nov 4, 2021

Ah did I make a mistake in implementing this in 3433a91. Could you mention this in the commit message?

The improved fix would be to specify the Environment kwargs so it clear what is being set to None e.g.:

Environment(search_path=plugin_dirs, platform=None, python=None)

@mhertz
Copy link
Contributor Author

mhertz commented Nov 5, 2021

Thanks alot for your reply and review Cas :) I'm sorry but stupidly deleted my forked repo and branch, and tried bring it back by pulling the PR and repushing, which worked but didn't update this here(noob at git and PRs in general), only mine, so I remade new PR and will close this, and obviously not do that mistake again. Sorry about that.

@mhertz mhertz closed this Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants