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

Make sydent.util pass mypy --strict #418

Merged
merged 12 commits into from
Oct 14, 2021
Merged

Make sydent.util pass mypy --strict #418

merged 12 commits into from
Oct 14, 2021

Conversation

DMRobertson
Copy link
Contributor

@DMRobertson DMRobertson commented Oct 8, 2021

Builds on #416 and #417.

@DMRobertson DMRobertson requested a review from a team October 14, 2021 12:01
@DMRobertson
Copy link
Contributor Author

I bumped the unpaddedbase64 version to pull in a version that has type hints. I don't think that's a problem, but would appreciate it if someone could confirm!

@callahad
Copy link
Contributor

I bumped the unpaddedbase64 version to pull in a version that has type hints. I don't think that's a problem, but would appreciate it if someone could confirm!

I did that upgrade; there are no functional or API changes, just typing, so the new constraint should be fine.

@babolivier babolivier self-assigned this Oct 14, 2021
sydent/util/__init__.py Show resolved Hide resolved
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