Skip to content

Update pre-commit config#2331

Merged
JelleZijlstra merged 6 commits into
psf:mainfrom
sadikkuzu:patch-1
Jun 22, 2021
Merged

Update pre-commit config#2331
JelleZijlstra merged 6 commits into
psf:mainfrom
sadikkuzu:patch-1

Conversation

@sadikkuzu
Copy link
Copy Markdown
Contributor

Update pre-commit config

via pre-commit autoupdate

Updating https://gitlab.com/pycqa/flake8
... updating 3.9.0 -> 3.9.2.
Updating https://github.com/pre-commit/mirrors-mypy
... updating v0.812 -> v0.902.
Updating https://github.com/pre-commit/mirrors-prettier
... updating v2.2.1 -> v2.3.1.

via `pre-commit autoupdate`

```
Updating https://gitlab.com/pycqa/flake8
... updating 3.9.0 -> 3.9.2.
Updating https://github.com/pre-commit/mirrors-mypy
... updating v0.812 -> v0.902.
Updating https://github.com/pre-commit/mirrors-prettier
... updating v2.2.1 -> v2.3.1.
```

Signed-off-by: SADIK KUZU <sadikkuzu@hotmail.com>
@JelleZijlstra JelleZijlstra added the ci: skip news Pull requests that don't need a changelog entry. label Jun 13, 2021
Comment thread Pipfile
flake8 = "*"
flake8-bugbear = "*"
mypy = ">=0.812"
types-dataclasses = ">=0.1.3"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This should only be needed <3.7, can we express that here?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Good call, will do that since IIRC it's possible.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

For the record, this is actually not possible due to python_version=3.6 in mypy.ini.

@ichard26 ichard26 marked this pull request as draft June 13, 2021 22:33
@ichard26
Copy link
Copy Markdown
Collaborator

Sorry for not updating until now. Your PR was mostly A-OK to land except for the fact that mypy 0.9XX introduces modular typeshed which means we need to pull in a few types-* packages. That is what I've been working on but I'm out of time today.

Thanks for the PR!

@sadikkuzu
Copy link
Copy Markdown
Contributor Author

@ichard26 ichard26 marked this pull request as ready for review June 22, 2021 00:52
@ichard26 ichard26 requested a review from JelleZijlstra June 22, 2021 00:59
@JelleZijlstra JelleZijlstra merged commit 3980b4b into psf:main Jun 22, 2021
@sadikkuzu
Copy link
Copy Markdown
Contributor Author

🚀

@sadikkuzu sadikkuzu deleted the patch-1 branch October 1, 2021 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: skip news Pull requests that don't need a changelog entry.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants