Skip to content

Conversation

@mikemhenry
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented May 17, 2022

Codecov Report

Merging #60 (8d39df4) into main (62ba1a7) will not change coverage.
The diff coverage is 85.71%.

@mikemhenry
Copy link
Collaborator Author

@IAlibay is there a reason we don't just run black to fix these errors? Or just add a pre-commit?

@IAlibay
Copy link
Member

IAlibay commented May 17, 2022

@IAlibay is there a reason we don't just run black to fix these errors? Or just add a pre-commit?

Richard kinda merged things eagerly before I could ask in the previous PR 🤣

I would say just fix the errors now and we can go pre-commit later?

@mikemhenry mikemhenry merged commit 0870e93 into main May 17, 2022
@mikemhenry mikemhenry deleted the mikemhenry-patch-1 branch May 17, 2022 18:44
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