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

Upgrade dependencies and use new pylint GitHub formatter #59

Merged
merged 4 commits into from
Sep 11, 2024

Conversation

Splines
Copy link
Collaborator

@Splines Splines commented Aug 14, 2024

This resolves #53. Related: pylint-dev/pylint#9443. See the release notes of version 3.2.0 of pylint:

A new github reporter has been added. This reporter returns the output of pylint in a format that Github can use to automatically annotate code. Use it with pylint --output-format=github on your Github Workflows.

Note that I didn't remove the plum dispatch dependency here, since this is done with the new API in #41. After having merged this, we resolve the merge conflict in #41.

Tip

You will have to run pipenv install afterwards since the lockfile was updated.

@Splines Splines added the CI/CD Continuous Integration / Continuous delivery label Aug 14, 2024
@Splines Splines requested a review from paul019 August 14, 2024 10:53
@Splines Splines self-assigned this Aug 14, 2024
paul019
paul019 previously approved these changes Aug 25, 2024
@Splines Splines merged commit ec7b986 into main Sep 11, 2024
2 checks passed
@Splines Splines deleted the deps/pylint-upgrade branch September 11, 2024 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Continuous Integration / Continuous delivery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade pylint to new version to use CI messages
2 participants