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

[REF]: update to pylint 3.0.0, drop support for py3.7 #474

Merged
merged 3 commits into from
Oct 10, 2023

Conversation

antonag32
Copy link
Contributor

Pylint 3.0.0 has been released and with it a bunch of fixes and improvements. This repository has been updated to depend on this new version.

BREAKING CHANGE: Pylint 3.0.0 drops support for Python 3.7 so this plugin no longer supports Python 3.7 as well.

Closes #473.

@antonag32 antonag32 marked this pull request as draft October 5, 2023 04:15
Pylint 3.0.0 has been released and with it a bunch of fixes and
improvements. This repository has been updated to depend on this new
version.

BREAKING CHANGE: Pylint 3.0.0 drops support for Python 3.7 so this
plugin no longer supports Python 3.7 as well.

Related OCA#473.
Pylint v3.0.0 changed some of its behaviour externally (to users) and
internally as well (for developers) by depending on a newer release of
astroid. Code has been updated to work with said changes and fix the
current test suite.

Specifically astroid has dropped support for visit_tryexcept and other
friends, replacing it by visit_try.

Pylint's invalid check no longer checks for a  minimum length of three
characters as well, so the expected messages had to be updated.

Closes OCA#473.
@antonag32 antonag32 marked this pull request as ready for review October 10, 2023 04:52
@antonag32
Copy link
Contributor Author

I think this could be merged, just wondering if we should merge this into main and create a 8.0 branch if any backport fixes are needed in the future. I like the idea and it seems that it was the same path taken when releasing version 8 (when lots of checks were moved to oca hooks).

requirements.txt Outdated
@@ -1,3 +1,3 @@
pylint-plugin-utils==0.7
pylint==2.15.*
pylint>=3, <4
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please, pin the pylint version in order to avoid adding new lints and creating new CI red without an explicit validation from our side and advice to our team and community

@moylop260
Copy link
Collaborator

moylop260 commented Oct 10, 2023

Please, consider the following extra steps:

  1. Create a new PR to pre-commit-vauxoo to use pylint-odoo 8.0.z for odoo <=13.0
    And using 9.0.z for odoo >= 14.0

  2. Create issues for our biggest projects with the output running the new pre-commit-vauxoo

  3. Send email to our team with these list of new checks detected, issues and how to fix them
    with a deadline to fix them before to raise non-green status in the CI (maybe 2 weeks)

@moylop260 moylop260 merged commit 1bb3c60 into OCA:main Oct 10, 2023
1 of 15 checks passed
@moylop260 moylop260 deleted the pylint3-anton branch October 10, 2023 21:17
@moylop260
Copy link
Collaborator

moylop260 pushed a commit that referenced this pull request Oct 10, 2023
Pylint 3.0.0 has been released and with it a bunch of fixes and
improvements. This repository has been updated to depend on this new
version.

BREAKING CHANGE: Pylint 3.0.0 drops support for Python 3.7 so this
plugin no longer supports Python 3.7 as well.

* update code to work with pylint v3

Pylint v3.0.0 changed some of its behaviour externally (to users) and
internally as well (for developers) by depending on a newer release of
astroid. Code has been updated to work with said changes and fix the
current test suite.

Specifically astroid has dropped support for visit_tryexcept and other
friends, replacing it by visit_try.

Pylint's invalid check no longer checks for a  minimum length of three
characters as well, so the expected messages had to be updated.

Closes #473.
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.

Migrate to pylint 3.0.0
2 participants