• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tarmolov / git-hooks-js
99%
master: 98%

Build:
Build:
LAST BUILD BRANCH: use-relative-paths-in-hooks
DEFAULT BRANCH: master
Repo Added 18 Jul 2015 02:12PM UTC
Files 2
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH use-relative-paths-in-hooks
branch: SELECT
CHANGE BRANCH
x
  • No branch selected
  • ignore-dir-and-non-exec
  • master
  • v1.1.10
  • v1.1.2
  • v1.1.3
  • v1.1.5
  • v1.1.6
  • v1.1.7
  • v1.1.8
  • v1.1.9

pending completion
174

Pull #62

travis-ci

web-flow
Use relative path to git-hooks in installed hooks scripts

Before this patch, installing git-hooks would use absolute paths to
require `node_modules/git-hooks/lib/git-hooks`. This works most of
the time, but there are instances in which it does not. For example:

I do development on a host machine, but yarn install runs inside a
Docker container. In this case, the absolute path to `node_modules`
is different in the two environments. This means for the git-hooks
module to work as intended, I am only able to run Git commands from
inside of the container. I typically run Git commands from the host
machine, so this doesn't work for me.

More importantly, though, I see no reason to force everybody working
on my project to use Git in one environment or the other.

After this patch, installed git-hooks use relative paths to require
`node_modules/git-hooks/lib/git-hooks`.
Pull Request #62: Use relative path to git-hooks in installed hooks scripts

1 of 1 new or added line in 1 file covered. (100.0%)

95 of 96 relevant lines covered (98.96%)

166.46 hits per line

Relevant lines Covered
Build:
Build:
96 RELEVANT LINES 95 COVERED LINES
166.46 HITS PER LINE
Source Files on master
Detailed source file information is not available for this build.

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
174 use-relative-paths-in-hooks Use relative path to git-hooks in installed hooks scripts Before this patch, installing git-hooks would use absolute paths to require `node_modules/git-hooks/lib/git-hooks`. This works most of the time, but there are instances in which it does no... Pull #62 26 Apr 2019 09:32PM UTC web-flow travis-ci pending completion  
173 master Always return full path to git dir Pull #57 12 Jun 2018 08:21AM UTC web-flow travis-ci pending completion  
172 master Test with real git dir (#58) * Test with real git dir New code fails with fake .git dir as it doesn't guess, it asks git. * Use tmp package to get temporary folder name * Fix typo in uninstall test It was running the install, not unin... push 11 Jun 2018 07:52PM UTC tarmolov travis-ci pending completion  
171 master Always return full path to git dir Pull #57 05 Jun 2018 04:28PM UTC web-flow travis-ci pending completion  
170 master Add test case for git repo not having a hooks folder Pull #58 05 Jun 2018 04:18PM UTC web-flow travis-ci pending completion  
168 master Use tmp package to get temporary folder name Pull #58 04 May 2018 08:38AM UTC web-flow travis-ci pending completion  
167 master Always return full path to git dir Pull #57 04 May 2018 08:36AM UTC web-flow travis-ci pending completion  
165 master Upgrade dev dependencies in package.json (#55) * Upgrade dev dependency chai * Upgrade dev dependency istanbul * Upgrade dev dependency jshint * Upgrade dev dependency mocha * Upgrade dev dependency jscs But stick with a 2.x.x bran... push 03 May 2018 06:19PM UTC tarmolov travis-ci pending completion  
164 master Use tmp package to get temporary folder name Pull #58 03 May 2018 05:14PM UTC web-flow travis-ci pending completion  
162 master Remove polyfill code for old node JS versions Pull #55 03 May 2018 01:18PM UTC web-flow travis-ci pending completion  
See All Builds (125)
  • Repo on GitHub
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc