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

chore(ci): fix lint job #1718

Merged
merged 10 commits into from
May 15, 2023
Merged

chore(ci): fix lint job #1718

merged 10 commits into from
May 15, 2023

Conversation

stegayet
Copy link
Contributor

@stegayet stegayet commented May 14, 2023

Follow-up on #1699.

This PR:

  • moves all lint steps to Python 3.10
  • upgrades pydocstyle to 6.3.0 to have Python 3.10 support
  • excludes some rules for pydocstyle to skip failures for now (let me know if you want to fix them in this PR)
  • fixes undetected mypy error

@stegayet stegayet mentioned this pull request May 14, 2023
@Nusnus Nusnus self-requested a review May 14, 2023 12:05
@stegayet stegayet marked this pull request as ready for review May 14, 2023 14:01
@stegayet
Copy link
Contributor Author

I’ve run last checks, this is ready for review 👍

setup.cfg Outdated
@@ -48,7 +48,7 @@ files =


[pep257]
ignore = D102,D104,D203,D105,D213
ignore = D102,D107,D104,D203,D105,D205,D212,D213,D401,D412,D406,D407,D411,D413,D417
Copy link
Member

Choose a reason for hiding this comment

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

are the failures too much to fix?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have to dig it into these issues but it seems to be only formatting issues, I can have a look.

Copy link
Contributor Author

@stegayet stegayet May 14, 2023

Choose a reason for hiding this comment

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

I fixed most of the rules, the other ones require some doc rewrite that can maybe be handled later ?

@stegayet stegayet force-pushed the chore/fix-lint-job branch 2 times, most recently from f66c0f8 to adf743a Compare May 14, 2023 17:18
@stegayet stegayet requested a review from auvipy May 14, 2023 18:23
@auvipy auvipy added this to the 5.3 milestone May 15, 2023
Copy link
Member

@auvipy auvipy left a comment

Choose a reason for hiding this comment

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

great job

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