Skip to content
This repository has been archived by the owner on Nov 25, 2022. It is now read-only.

Enable mypy typechecking & isort checking for Sygnal #73

Merged
merged 4 commits into from
Jul 16, 2020

Conversation

reivilibre
Copy link
Contributor

@reivilibre reivilibre commented Jul 2, 2020

Not ready yet; waiting on matrix-org/sygnal#131.

Now also uses tox.

Signed-off-by: Olivier Wilkinson (reivilibre) olivier@librepush.net

Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
@reivilibre reivilibre changed the title Enable mypy typechecking for Sygnal Enable mypy typechecking & isort checking for Sygnal Jul 3, 2020
@clokep
Copy link
Member

clokep commented Jul 10, 2020

@reivilibre Now that we merged the underlying PR is this ready to go?

@reivilibre reivilibre marked this pull request as ready for review July 14, 2020 10:09
@reivilibre
Copy link
Contributor Author

I think the isort CLI changed, going to check...

…ility

Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
@reivilibre reivilibre requested a review from a team July 14, 2020 10:16
sygnal/pipeline.yml Outdated Show resolved Hide resolved
sygnal/pipeline.yml Outdated Show resolved Hide resolved
@reivilibre
Copy link
Contributor Author

Sure, I'll look into it!

Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>
@reivilibre reivilibre requested a review from a team July 15, 2020 10:56
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

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

lgtm

@richvdh richvdh merged commit 94fe998 into master Jul 16, 2020
@richvdh richvdh deleted the rei/sygnal_mypy branch July 16, 2020 13:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants