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

Integrate Prettier #41

Closed
koistya opened this issue Jun 26, 2017 · 4 comments
Closed

Integrate Prettier #41

koistya opened this issue Jun 26, 2017 · 4 comments

Comments

@koistya
Copy link
Member

koistya commented Jun 26, 2017

https://prettier.github.io/prettier/


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@pruhstal
Copy link

Wouldn't it make more sense to enforce code styling with some eslint rules and not accept anything that doesn't pass a pre-commit hook into this repo?

Seems like there are more things that would be useful to add to this particular project.

@cmmartin
Copy link
Contributor

cmmartin commented Jun 28, 2017

I personally prefer prettier to eslint, although they can both be used as mentioned in this tweet. This project is already using eslint. The prettier docs include an example of using it alongsideeslint.

koistya referenced this issue in koistya/relay-starter-kit Aug 3, 2017
@followbl
Copy link

@koistya close this one?

@koistya
Copy link
Member Author

koistya commented Aug 17, 2017

@followbl yep, thanks for heads up!

@koistya koistya closed this as completed Aug 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants