sds / overcommit / 1937 / 3
91%
master: 91%

DEFAULT BRANCH: master
Ran
Files 207
Run time 45s
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
coverage: 90.672% (+0.01%) from 90.658%
2.6

push

travis-ci

web-flow 
Update ReplaceBranch hook to avoid running on `--amend` by default (#725)

* change default branch pattern for ReplaceBranch

This changes the default branch pattern to a simpler pattern.

This is added to make understanding the hook and it's usage a bit
easier.

* change an example in ReplaceBranch spec to default

This changes the spec to test against the same default branch
replacement pattern specified in the default config.

This is just to make things more consistent.

* improve class comment describing ReplaceBranch usage

This makes the usage of this (hopefully) obvious to anyone reading it.

* add `commit` to default ReplaceBranch skipped types

This makes the default behavior of this hook skip commits that explicitly
give the SHA, i.e, `--amend`, `-c`, or `-C`.

This is added because most people (I'd argue) don't want to run this
hook on `amend`s.

* remove extra hard-coded space in ReplaceBranch

This removes a trailing space after the repacement text, as well as
`#strip`ing the replacement text.

* refactor ReplaceBranch specs to avoid so much mocking

This changes the specs for the ReplaceBranch hook to

- test the contents of the file that `git` writes it's commit message to
- explicitly set config variables instead of mocking them

* update ReplaceBranch specs to test all `skipped_commit_types`

* shorten a long line for the linter

3091 of 3409 relevant lines covered (90.67%)

137.34 hits per line

Source Files on job 1937.3 (2.6)