question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Enable third-party online linters

See original GitHub issue

Current tests cover only some functionality, proving it works. But because of python’s dynamic nature, there are bunch of problems, which are commonly missed.

I suggest enabling linters, which are being triggered on each commit or PR additionally to Travis. Personally I’ve used codacy and landscape. I’m satisfied with their features and propose enabling both or one of them.

For example, look at these issues codacy pointed out recently: https://www.codacy.com/app/webknjaz/cherrypy/file/3463172348/issues/source?bid=3283917&fileBranchId=3283917#l1430 There’s misuse of NotImplementedError and missing param while calling method among other errors, which noone mentioned before.

Among other features, there’s an optional possibility to enable adding comments upon checks finish to PRs/commits, which could be very useful.

/cc: @jaraco

Issue Analytics

  • State:open
  • Created 7 years ago
  • Comments:7 (7 by maintainers)

github_iconTop GitHub Comments

1reaction
jaracocommented, Dec 25, 2016

It’s also possible now to run tox -e linter to run the linters.

1reaction
webknjazcommented, Aug 21, 2016

Codacy allows disabling some of errors if needed. Regarding tests @ Travis, there’s this cool utility tool called pre-commit which I use for a while. It lets users nicely organize common checks and can be added locally as pre-commit or pre-push script, running preconfigured hooks on changed files only. Also it can be used as one of CI checks.

For now, I suggest enabling just codacy to see what we’ll get 😃

Read more comments on GitHub >

github_iconTop Results From Across the Web

Linting Python in Visual Studio Code
To enable linters, open the Command Palette (Ctrl+Shift+P) and select the Python: Select Linter command. The Select Linter command adds "python.linting.<linter> ...
Read more >
Stylelint: The Style Sheet Linter We've Always Wanted
The third strategy is to go over all of the rules and configure every one. I prefer this strategy because I want to...
Read more >
Linters aren't in your way. They're on your side
Traditionally, linters make sure your code is clean and easy for teammates to read. They check for errors, bugs, style, and more.
Read more >
What Is a Linter? Here's a Definition and Quick-Start Guide
What is a linter? In short, a linter is a tool to help you improve your code. The concept of linter isn't exclusive...
Read more >
Linters | golangci-lint
Name Description Presets Since asasalint ⚙️ check for pass any as any in variadic func(...any) bugs 1.47.0 bidichk ⚙️ Checks for dangerous unicode character sequences...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found