-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature/refactor formatter #111
Conversation
f97efba
to
2ed2e36
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine.
ea8bc1e
to
47ecd0c
Compare
d93e6ea
to
81f7113
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost perfect, just two minor things 😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
One thing that I think might be important is that as soon as this is merged, everybodies pre-commit
hooks will stop working. It might be good idea to send a chat message to everybody informing them about that and what to do that they start working again
a674229
to
e029aa8
Compare
LGTM 😃 |
39f36f4
to
ebbf8f8
Compare
No description provided.