search

rycus86 / githooks

Githooks: per-repo and global Git hooks with version control
MIT License
381 stars 20 forks source link

Feature: Maintain only server hooks :heart: #64

Closed gabyx closed 4 years ago

gabyx commented 4 years ago

To issue #63

coveralls commented 4 years ago

Pull Request Test Coverage Report for Build 470

Changes Missing Coverage Covered Lines Changed/Added Lines %
install.sh 15 18 83.33%
<!-- Total: 16 19 84.21% -->
Totals Coverage Status
Change from base Build 462: 0.2%
Covered Lines: 1750
Relevant Lines: 2006
💛 - Coveralls
rycus86 commented 4 years ago

Could you please add a quick test for this as well to check that the expected hooks are getting installed? Thanks!

gabyx commented 4 years ago

Review changes. Added some more hooks. Added a test. Will fix tonight if it does not work...

gabyx commented 4 years ago

Added a test. Everything works. Read to merge :-) Thx

gabyx commented 4 years ago

Haha, the test was wrong, hui...

gabyx commented 4 years ago

Ok, now its good to go, The test was wrong, thats why I did not see the missing $HOOK_NAMES...

rycus86 commented 4 years ago

OK, this looks good, just resolve the conflicts please.