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.

Sign up for GitHub

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

Jump to bottom

No prompt displayed to install pylint #5087

Closed
DonJayamanne opened this issue Apr 2, 2019 · 2 comments
Closed

No prompt displayed to install pylint #5087

DonJayamanne opened this issue Apr 2, 2019 · 2 comments
Assignees
@karrtikr
Labels
area-linting bug Issue identified by VS Code Team member as probable bug regression Bug didn't exist in a previous release
Milestone
2019, week 15 - A...

Comments

@DonJayamanne
Copy link

  • Create a venv
  • Select a pylint as the linter using the command Select Linter
  • Start making changes to files, edit, save... no linting takes place.

There are prompts are displayed asking the user to install pylint
Errors are logged in console, but no logs.

This is specific to pylint.
Most probably a bug with the special handling of pylint when language server is turned on.
@ghost ghost added the triage-needed Needs assignment to the proper sub-team label Apr 2, 2019
@DonJayamanne DonJayamanne added area-linting needs PR regression Bug didn't exist in a previous release bug Issue identified by VS Code Team member as probable bug labels Apr 2, 2019
@ghost ghost removed triage-needed Needs assignment to the proper sub-team labels Apr 2, 2019
@karrtikr karrtikr added important Issue identified as high-priority P0 and removed important Issue identified as high-priority labels Apr 10, 2019
@DonJayamanne
Copy link
Author

Most probable location of bug: https://github.com/Microsoft/vscode-python/blob/master/src/client/linters/linterAvailability.ts

@karrtikr karrtikr mentioned this issue Apr 18, 2019
Merged
4 tasks
@ericsnowcurrently
Copy link
Member

I was able to verify that, after picking pylint from the "Select Linter" command, I was prompted to install pylint.

@ghost ghost removed the needs PR label Apr 22, 2019
@ericsnowcurrently ericsnowcurrently removed their assignment Apr 22, 2019
@lock lock bot locked as resolved and limited conversation to collaborators May 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@karrtikr karrtikr

Labels
area-linting bug Issue identified by VS Code Team member as probable bug regression Bug didn't exist in a previous release
Projects
None yet
Milestone
2019, week 15 - April Sprint 8
Development

No branches or pull requests
3 participants
@ericsnowcurrently @DonJayamanne @karrtikr