Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

fix Windows compatablity for commiting#339

Closed
nschonni wants to merge 2 commits into
nodejs:masterfrom
nschonni:husky-windows
Closed

fix Windows compatablity for commiting#339
nschonni wants to merge 2 commits into
nodejs:masterfrom
nschonni:husky-windows

Conversation

@nschonni

@nschonni nschonni commented Oct 11, 2019

Copy link
Copy Markdown
Member

Description

Working on a Windows machine for the previous PR, I was unable to push a branch because of the pre-push hook. Tracked down 2 known issues related to Windows with the current config and was able to push this without commenting out the hook for this.

Related Issues

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #339 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #339   +/-   ##
=======================================
  Coverage   39.92%   39.92%           
=======================================
  Files          22       22           
  Lines         273      273           
  Branches       62       62           
=======================================
  Hits          109      109           
  Misses        162      162           
  Partials        2        2

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 42f53a6...63522d2. Read the comment docs.

@nschonni nschonni changed the title chore: Replace deprecated commitlint param fix Windows compatablity for commiting Oct 11, 2019
@nschonni

Copy link
Copy Markdown
Member Author

Closing since #323 landed

@nschonni nschonni closed this Oct 14, 2019
@nschonni nschonni deleted the husky-windows branch October 16, 2019 18:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants