Test the merge in a build initiated by a pull request
Are builds initiated by "pull requests" merged before the build? It looks like they aren't.
In others build systems, the build is done using the result of the merge between origin and the upstream branch.
Comments are currently closed for this discussion. You can start a new one.
Keyboard shortcuts
Generic
| ? | Show this help |
|---|---|
| ESC | Blurs the current field |
Comment Form
| r | Focus the comment reply box |
|---|---|
| ^ + ↩ | Submit the comment |
You can use Command ⌘ instead of Control ^ on Mac
Support Staff 1 Posted by Feodor Fitsner on 22 Jan, 2015 09:06 PM
Yeah, AppVeyor fetches virtual merge commit created by GitHub and you should see these two additional commands in the build log:
2 Posted by Diogo Mafra on 23 Jan, 2015 11:58 AM
Right, I see this command in the logs.
But, in a pull request that can't be automatically merged, it built using the commit ("git checkout -qf xyz123..."). Is this the expected thing? I thought the build would fail, as it can't merge the source.
Support Staff 3 Posted by Feodor Fitsner on 23 Jan, 2015 02:45 PM
No, it's not expected. Are you sure it was triggered by PR webhook event? You may check "Recent deliveries" of project webhook in GitHub to see what was request and AV response. Also send me AV project name and build version to check in database.
Yesterday we deployed an update that should reliably prevent starting builds for non-mergeable PRs.
-Feodor
4 Posted by Diogo Mafra on 23 Jan, 2015 03:10 PM
Looks like this update corrected the problem. Now it is working as expected.
Thank you.
Support Staff 5 Posted by Feodor Fitsner on 23 Jan, 2015 03:11 PM
Great, thanks for the update!
-Feodor
Ilya Finkelshteyn closed this discussion on 25 Aug, 2018 01:53 AM.