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.

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

Use PyLint 3.0+ #672

Merged
merged 3 commits into from
Oct 7, 2024
Merged

Use PyLint 3.0+ #672

merged 3 commits into from
Oct 7, 2024

Conversation

sbrunner
Copy link
Member

@sbrunner sbrunner commented Oct 5, 2024

Based on #671, fix some errors

Copy link
Collaborator

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not try to just vendor the missing deprecated function in prospector yet, did you think about it ?

@sbrunner
Copy link
Member Author

sbrunner commented Oct 5, 2024

When I saw the cleaning pull request, the missing function uses a lot of deprecated, then removed objects, then I didn't think that's easy to do!

@sbrunner sbrunner force-pushed the test-pylint-3.0-test branch 2 times, most recently from 86ff5ac to 5097b66 Compare October 6, 2024 07:43
@sbrunner sbrunner changed the title Adding PyLint 3.0+ back bis Use PyLint 3.0+ Oct 7, 2024
@sbrunner sbrunner force-pushed the test-pylint-3.0-test branch 2 times, most recently from 17de0a4 to 971dc91 Compare October 7, 2024 06:31
@sbrunner sbrunner force-pushed the test-pylint-3.0-test branch from 971dc91 to 683f0b3 Compare October 7, 2024 07:02
@@ -91,6 +91,8 @@ def test_will_throw_useless_suppression(self):

found_files = _get_test_files("testpath", "testpath/test_useless_suppression.py")
pylint_tool.configure(config, found_files)
# useless-suppression is now disable by default in pylint
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sbrunner sbrunner force-pushed the test-pylint-3.0-test branch from 683f0b3 to 359e573 Compare October 7, 2024 07:45
@sbrunner
Copy link
Member Author

sbrunner commented Oct 7, 2024

@Pierre-Sassoulas @carlio Do you think that's ready to be merged?

As I currently understand, if we want a better (more direct) implementation, we should do some modifications directly in PyLint!

Copy link
Collaborator

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great !

Copy link
Member

@carlio carlio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@Pierre-Sassoulas Pierre-Sassoulas merged commit 5287be4 into master Oct 7, 2024
6 checks passed
@Pierre-Sassoulas Pierre-Sassoulas deleted the test-pylint-3.0-test branch October 7, 2024 08:47
@sbrunner
Copy link
Member Author

sbrunner commented Oct 7, 2024

Thank you :-)

Do we want something other in version 1.12 or do we do the release?

@carlio
Copy link
Member

carlio commented Oct 7, 2024

I think it's best to release it now as people are waiting for a Python 3.12 compatible version; any other bugfixes or cleanups can go into a patch version.

I am pretty sure the PyPI token secrets in the repo are still correct so the GitHub action can still be used to make the release, if you want to.

@Pierre-Sassoulas
Copy link
Collaborator

You need to push a tag then create a github release from the tag to release on pypi @sbrunner (I could do it if you want, but it's a good thing if you already did it once imo)

@carlio
Copy link
Member

carlio commented Oct 7, 2024

https://github.com/prospector-dev/prospector/blob/master/pyproject.toml#L3 <-- need to update that :)

@sbrunner
Copy link
Member Author

sbrunner commented Oct 7, 2024

It's on release creation => Done :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants