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.

Sign up for GitHub

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

Jump to bottom

Apply yapf on the whole repo #141

Closed
rafonseca opened this issue Feb 6, 2019 · 2 comments
Closed

Apply yapf on the whole repo #141

rafonseca opened this issue Feb 6, 2019 · 2 comments

Comments

@rafonseca
Copy link
Contributor

https://github.com/google/yapf/

It is a python formatter. If we always use it, we will have a unified style and reviewing PRs will be much easier.
@rafarui
Copy link
Collaborator

rafarui commented Feb 6, 2019

I like the idea. Today I use refactor from pycharm to auto fix when I remember.

@rafonseca
Copy link
Contributor Author

We should apply after next release. If we apply before, the PRs merges will be a nightmare.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rafonseca @rafarui