Skip to content

Add error end column to the output then add the information in all the checkers #1590

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

Closed
purpleP opened this issue Jul 24, 2017 · 6 comments
Closed
Labels
Enhancement ✨ Improvement to a component Hacktoberfest High effort 🏋 Difficult solution or problem to solve
Milestone

Comments

@purpleP
Copy link

purpleP commented Jul 24, 2017

Right now it's hard to use pylint as linter integrated into text editor, because it isn't showing where error ends and sometimes doesn't even show where error starts (shows zero column for example in unused import)

@PCManticore
Copy link
Contributor

You can customize this by using the --msg-template parameter. How each editor integration is using it though is not something that pylint should be aware of.

@purpleP
Copy link
Author

purpleP commented Jul 24, 2017

@PCManticore According to docs you only have column field. Which is the column where error starts. I'm talking about outputting where the error ends.

@purpleP
Copy link
Author

purpleP commented Jul 28, 2017

@PCManticore The issue isn't resolved. Please reopen.

@pappasam
Copy link

pappasam commented Sep 3, 2021

I agree, this issue should be re-opened

@Pierre-Sassoulas Pierre-Sassoulas added Enhancement ✨ Improvement to a component Help wanted 🙏 Outside help would be appreciated, good for new contributors High effort 🏋 Difficult solution or problem to solve labels Sep 3, 2021
@Pierre-Sassoulas Pierre-Sassoulas changed the title Show error end column. Add error end column to the output then add the information in all the checkers Sep 3, 2021
@Pierre-Sassoulas
Copy link
Member

This is a huge one though. By the way I read that linked issue in dense analysis and highlighing till the end of the line should indeed be prefered. This can be the default value until we manage to add the real information in all the checkers.

@Pierre-Sassoulas
Copy link
Member

Closing as completed because it was done in #5336

@Pierre-Sassoulas Pierre-Sassoulas removed Breaking changes for 3.0 🦤 Help wanted 🙏 Outside help would be appreciated, good for new contributors labels Jul 5, 2022
pappasam added a commit to pappasam/config that referenced this issue Jul 5, 2022
@Pierre-Sassoulas Pierre-Sassoulas modified the milestones: 3.0.0, 3.0.0a6 Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ✨ Improvement to a component Hacktoberfest High effort 🏋 Difficult solution or problem to solve
Projects
None yet
Development

No branches or pull requests

5 participants