default to importStrategy fromEnvironment if pylint is in available #558
davetapley
started this conversation in
Ideas
Replies: 2 comments
-
Neutral on this. I think experts can find and change the option 'easily' but non-experts / newcomers can easily shoot themself in the foot with a bad env and an incompatible pylint. Maybe it's a good thing that the default protect newcomers from 'breaking' the plugin. |
Beta Was this translation helpful? Give feedback.
0 replies
-
I agree with @Pierre-Sassoulas . We have seen new users struggle with env issues, hence the default to run with bundled. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I was a bit confused just now because I bumped to the new pylint 3.2, but was still seeing an erroneous error (from pylint-dev/astroid#1015).
Only after some digging did I realize the extension comes bundled with a
pylint
version.I see I can
importStrategy
fromEnvironment
, which is fine as a workaround, but it made me think:Would a PR be welcome to change the behavior to default to
importStrategy
fromEnvironment
ifpylint
is in available in the current environment? I can only assume that's what 99.99% of people would expect.Beta Was this translation helpful? Give feedback.
All reactions