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

Add Prettier #105

Merged
merged 3 commits into from
Dec 20, 2018
Merged

Add Prettier #105

merged 3 commits into from
Dec 20, 2018

Conversation

TimonVS
Copy link
Member

@TimonVS TimonVS commented Dec 20, 2018

While working on this awesome tool I was really missing auto formatting. In this PR I've added Prettier and I added a pre-commit hook so that all commits will be consistently formatted. I also removed standard as it clashes with the formatting of Prettier, if desired I can add back standard by utilizing eslint-config-prettier which will disable all rules that clash with Prettier.

Let me know if you'd like me to change anything to suit your needs best :)

If you intend to merge this PR you can run yarn prettier afterwards to format all existing files.

@toolmantim toolmantim merged commit 6c8cd43 into release-drafter:master Dec 20, 2018
@toolmantim
Copy link
Collaborator

Thanks @TimonVS! Happy to switch to Prettier. I've just applied the yarn prettier changes and merged 🙌🏼

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.

None yet

2 participants