Skip to content

Conversation

@samer-hamood
Copy link
Contributor

@samer-hamood samer-hamood commented Jan 25, 2025

Pylint could not consistently run on Python 3.12 and 3.13, throwing errors the previous versions weren't. This prevented testing in CI for the later Python versions.

Upgraded Pylint to 3.3.3, adjusting the GitHub Action 'build' workflow to support Python 3.12 and 3.13 and disabling lint errors not visible with the previous versions of Python. This version of Pylint requires the minimum Python version to be 3.9. It's worth noting that Python 3.8 has reached EOL (See versions).

@samer-hamood samer-hamood force-pushed the upgrade_pylint_to_support_python_3.12_and_3.13 branch from edeb77b to d16ed5f Compare January 25, 2025 17:11
@samer-hamood samer-hamood marked this pull request as draft January 27, 2025 17:53
@samer-hamood samer-hamood marked this pull request as ready for review January 27, 2025 18:20
@samer-hamood samer-hamood mentioned this pull request Jan 27, 2025
2 tasks
Copy link
Owner

@EntilZha EntilZha left a comment

Choose a reason for hiding this comment

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

Overall LGTM, I think its fine to drop 3.8 to simplify the logic. Will merge once that's done

@samer-hamood samer-hamood force-pushed the upgrade_pylint_to_support_python_3.12_and_3.13 branch from 5a01a3b to c93e4f4 Compare January 27, 2025 21:25
@samer-hamood
Copy link
Contributor Author

I think build (3.8) may need to be removed from master branch

@EntilZha EntilZha merged commit 316a194 into EntilZha:master Jan 28, 2025
5 checks passed
@EntilZha
Copy link
Owner

I just bypassed the 3.8 check since its going away :)

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.

2 participants