Wrong commit sha used for PRs - Githubissues

miguelfrde / travieso

Travis CI build job status directly on GitHub!

http://miguelfrde.github.io/travieso

MIT License

14 stars 1 forks source link

Wrong commit sha used for PRs #10

Closed jarekwg closed 7 years ago

jarekwg commented 8 years ago

Steps to reproduce

Have Travis CI tests run on a PR. It runs two tests by default: in-branch and merged.
Notice that the commit sha for the merged test is not the same as the HEAD of the branch that is to be merged. Using Github lingo, it uses merge_commit_sha instead.
Notice that as a result of the wrong commit sha being used, Travieso does not report the status back to the PR, because it's tying the status to the merge commit instead.

Expected output: Jobs for merge tests showing up on PR.

Actual output: Jobs for merge tests don't show up on PR.

Metadata

Travieso version:: ae3a069f3fb2f3e7fed8839c8dfc17fc4df07e70

Python version:: 3.5.1

Operating system:: Mac