1
72%
master: 72%

Ran 28 Nov 2018 04:10PM UTC

Files 466

Run time 31s

Badge

Embed ▾

Committed 28 Nov 2018 04:10PM UTC coverage: 72.066% (+0.06%) from 72.006%

Job # 283.1

Build Type

push

jenkins

Committed by GitHub

Commit Message

Use ${sha1} instead of ${ghprbActualCommit} for Pre-commit Jobs

Descriptions of the changes in this PR:

### Motivation

According to  the documentation of the GitHub Pull Request plugin:
- ${sha1} means a commit which is the result of merging the PR branch with the target branch
- ${ghprbActualCommit} is directly the PR branch

So currently we are not testing the PR branch against the target branch, but only the branch "as it is"

see:
https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=4&ved=2ahUKEwje0PjpnuDeAhUF2SwKHXN4BgYQFjADegQIAxAB&url=https%3A%2F%2Fwiki.jenkins.io%2Fdisplay%2FJENKINS%2FGitHub%2Bpull%2Brequest%2Bbuilder%2Bplugin&usg=AOvVaw2XvPL5ynRYnqVEWqpe9cs1

### Changes

Use ${sha1} instead of ${ghprbActualCommit} for Pre-commit Jobs




Reviewers: Ivan Kelly <ivank@apache.org>, Sijie Guo

This closes #1818 from eolivelli/fix/jenskin-use-sha1

Run Details

25365 of 35197 relevant lines covered (72.07%)

0.72 hits per line

apache / bookkeeper / 283 / 1
72%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 283.1

apache / bookkeeper / 283 / 1 72% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 283.1

apache / bookkeeper / 283 / 1
72%
master: 72%

README BADGES
x