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, use Husky and lint-staged to auto-commit prettification #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joshwcomeau
Copy link
Owner

Yay Prettier support! No more debates about code convention :D

As suggested by @morajabi, I used Husky and lint-staged to auto-run Prettier on files before committing. As such, a buncha stuff was "prettified".

@morajabi you'll likely hit a bunch of merge conflicts if this gets merged in before your two PRs. I didn't actually change any of the code, though, so you can take your changes over these for everything. Prettier will re-run when you commit so there's no problem :)

Closes #10

},
"devDependencies": {
"husky": "0.14.3",
"lint-staged": "4.2.3"
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I re-ordered some stuff here, because I like for the dependencies to be near the top.

@jelly8173
Copy link

So is this usefull utility package or a playground for testify cool tricks?
Sure, Prettier resolves style issues but it brings dependency hell for not so expirienced programmers:
see #19

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.

Prettier
2 participants