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

Improve typing #516

Merged
merged 1 commit into from
Jun 21, 2024
Merged

Improve typing #516

merged 1 commit into from
Jun 21, 2024

Conversation

nikteliy
Copy link
Contributor

@nikteliy nikteliy commented May 12, 2024

  • add protocols for snap7's functions
  • add --strict parameter to mypy in pre-commit.conf

@nikteliy nikteliy force-pushed the improve_typing branch 2 times, most recently from 7c6f21e to cbd2bfe Compare May 13, 2024 17:05
@haronaut
Copy link

This PR will break python 3.8 usage (again).
see:
#517

@nikteliy nikteliy force-pushed the improve_typing branch 3 times, most recently from 60881b1 to 1662069 Compare May 15, 2024 14:56
@nikteliy nikteliy changed the title Improve typing [WIP] Improve typing May 15, 2024
@nikteliy nikteliy force-pushed the improve_typing branch 3 times, most recently from 21b133f to 3741002 Compare May 28, 2024 17:03
@nikteliy nikteliy changed the title [WIP] Improve typing Improve typing May 28, 2024
@gijzelaerr
Copy link
Owner

This PR will break python 3.8 usage (again). see: #517

and again the answer is that we dropped support for <=3.8

snap7/client/__init__.py Outdated Show resolved Hide resolved
.pre-commit-config.yaml Outdated Show resolved Hide resolved
snap7/client/__init__.py Show resolved Hide resolved
snap7/client/__init__.py Show resolved Hide resolved
snap7/client/__init__.py Show resolved Hide resolved
snap7/client/__init__.py Outdated Show resolved Hide resolved
snap7/server/__main__.py Outdated Show resolved Hide resolved
snap7/server/__init__.py Show resolved Hide resolved
snap7/util/db.py Show resolved Hide resolved
snap7/client/__init__.py Outdated Show resolved Hide resolved
@nikteliy nikteliy force-pushed the improve_typing branch 4 times, most recently from 3a96d04 to ccefa63 Compare June 3, 2024 16:39
@gijzelaerr gijzelaerr merged commit ac9110a into gijzelaerr:master Jun 21, 2024
59 checks passed
@gijzelaerr gijzelaerr added this to the 2.0 milestone Jul 5, 2024
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.

3 participants