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

chore(htmlhint) feature/add commitizen package, husky and prettier #287

Merged
merged 5 commits into from
Sep 3, 2018

Conversation

thedaviddias
Copy link
Member

Fixes: #255 & #258

  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.

Short description of what this resolves:

Add precommit hooks to ensure the uniformity of commit messages and prettier to format the code automatically.

Proposed changes:

  • Add commitizen and commitlint the same format for commits messages
  • Add husky to facilitate the creation of hooks
  • Add prettier to format the code automatically

@thedaviddias thedaviddias added the enhancement Functionality that enhances existing features label Sep 3, 2018
@codecov-io
Copy link

codecov-io commented Sep 3, 2018

Codecov Report

Merging #287 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           develop   #287   +/-   ##
======================================
  Coverage      100%   100%           
======================================
  Files            1      1           
  Lines            1      1           
======================================
  Hits             1      1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d51803f...b65b168. Read the comment docs.

@thedaviddias thedaviddias merged commit 1eba3bc into develop Sep 3, 2018
@thedaviddias thedaviddias mentioned this pull request Sep 3, 2018
2 tasks
thedaviddias added a commit that referenced this pull request Sep 3, 2018
)

* style(htmlhint): update name changes to changelog

* feat(htmlhint): add husky, lint-staged, commitizen and commilint

solve #258

* chore(htmlhint): remove lint-staged and move pretty-quick to precommit

* fix(htmlhint): remove grunt-cli and npm test

* chore(htmlhint): update package-lock
.prettierrc.json Show resolved Hide resolved
"analysis",
"javascript"
],
"author": {
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm missing the "author" and "contributors" :(

Copy link
Member Author

Choose a reason for hiding this comment

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

Don't see why, the list of contributors is given by Github easily and everyone can see it.

Copy link
Contributor

Choose a reason for hiding this comment

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

Because you can see the package.json in node_modules
(It's only for author and main contributors)

@thedaviddias thedaviddias deleted the feature/add-commitizen-package branch September 3, 2018 11:18
thedaviddias added a commit that referenced this pull request Sep 3, 2018
)

* style(htmlhint): update name changes to changelog

* feat(htmlhint): add husky, lint-staged, commitizen and commilint

solve #258

* chore(htmlhint): remove lint-staged and move pretty-quick to precommit

* fix(htmlhint): remove grunt-cli and npm test

* chore(htmlhint): update package-lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Functionality that enhances existing features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants