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

Weird recursion when upgrading from pylint-pytest < 1.1.0 with pylint 2 to their latest versions #61

Closed
webknjaz opened this issue Apr 24, 2024 · 3 comments
Closed

Weird recursion when upgrading from pylint-pytest < 1.1.0 with pylint 2 to their latest versions #61

webknjaz opened this issue Apr 24, 2024 · 3 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@webknjaz
Copy link

webknjaz commented Apr 24, 2024 •
edited
Loading

Describe the bug

Here's what I'm doing: cherrypy/cheroot@f109e85#diff-63a9c44a44acf85fea213a857769990937107cf072831e1a26808cfde9d096b9L270

To Reproduce

See cherrypy/cheroot#698. Getting the repo and running tox -e pre-commit -- pylint --all-files -vv

Expected behavior

If there's something wrong with the configuration, the error message should state that instead of spitting out tracebacks...

Additional context

I'm trying to upgrade some old linter pins to as new as I can. pylint is wrapped with pre-commit which runs on pre-commit.ci and the normal CI is also running it through tox.
@webknjaz webknjaz added the bug Something isn't working label Apr 24, 2024
@webknjaz
Copy link
Author

Is this #56? I'll try the suggestion to use an alpha release and report back.

@webknjaz
Copy link
Author

Is this #56? I'll try the suggestion to use an alpha release and report back.

Yep, using pylint-pytest ~= 2.0.0a0 fixes it. I'll mark it as resolved and hopefully other people will be able to locate the closed issue when debugging...

webknjaz added a commit to webknjaz/cheroot that referenced this issue Apr 24, 2024
@webknjaz
🧪 Bump pylint to v3.1.0
ed331d1 
This version supports Python 3.12 officially. As a part of this patch,
`pylint-pytest` is also updated with the bugfixes needed to stop
capping it.

An alpha version of the plugin is used due to a bug when running
`pylint` in parallel mode [[1]].

[1]: pylint-dev/pylint-pytest#61
@stdedos
Copy link
Collaborator

stdedos commented Apr 24, 2024

Duplicate of #56

I have one pesky issue that I'd like to fix before v2 - and sadly I haven't "meaningfully" contributed time towards https://github.com/pylint-dev/pylint-pytest for (more than) a couple of months

@stdedos stdedos closed this as not planned Won't fix, can't repro, duplicate, stale Apr 24, 2024
@stdedos stdedos added the duplicate This issue or pull request already exists label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stdedos @webknjaz