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
pylintversion.I see I can
importStrategyfromEnvironment, which is fine as a workaround, but it made me think:Would a PR be welcome to change the behavior to default to
importStrategyfromEnvironmentifpylintis 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