283
72%

Ran 28 Nov 2018 04:10PM UTC

Jobs 1

Files 466

Run time 32s

Badge

Embed ▾

pending completion

Build # 283

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

Jobs

ID	Job ID	Ran	Files	Coverage
1	283.1	28 Nov 2018 04:10PM UTC	0	72.07

Source Files on build 283

Detailed source file information is not available for this build.

apache / bookkeeper / 283
72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 283

apache / bookkeeper / 283 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 283

apache / bookkeeper / 283
72%

README BADGES
x