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

build: Add pre-commit hooks, prettier, use node 8 #20

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

billyvg
Copy link
Member

@billyvg billyvg commented Jul 24, 2019

  • Adds pre-commit hooks
  • Adds prettier
  • Re-format code
  • Upgrades some dev dependencies (eslint, mocha, babel)
  • Use node@8 in travis

billyvg added 2 commits July 24, 2019 10:38
* Adds pre-commit hooks
* Adds prettier
* Re-format code
* Upgrades some dev dependencies (eslint, mocha, babel)
@billyvg billyvg changed the title build: Add pre-commit hooks and prettier build: Add pre-commit hooks, prettier, use node 8 Jul 24, 2019
@billyvg billyvg force-pushed the build/add-precommit-prettier branch from ef28461 to 5e06675 Compare July 24, 2019 17:50
@billyvg
Copy link
Member Author

billyvg commented Jul 24, 2019

Since I was doing a bunch of cleanup.... thoughts on this @willdurand, @woutervanvliet?

Copy link
Contributor

@willdurand willdurand left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

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.

2 participants