Withdrawn: 8258477: Pre-submit testing using GitHub Actions should merge changes from target branch

duke duke at openjdk.java.net
Wed Apr 21 13:34:36 UTC 2021


On Wed, 16 Dec 2020 10:47:50 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

> Normally when running GitHub Actions on a pull request, what is checked out is the merge of the pull request with the latest changes on the target branch. This ensure that what is tested is as close as possible to what will actually be the result of integrating said pull request. 
> 
> In our use of GitHub Actions, we don't run directly on pull requests but instead allow contributors to run them in their own personal forks. In that context, there is no notion of a target branch. However, we can infer the target project and branch by encoding this in the workflow file. This allows us to perform the same merge manually, and achieve the same behaviour. 
> 
> The change also adds an option to disable this automated merge when launching the workflow manually, which could be useful when investigating unexpected test failures.
> 
> Note that downstream projects picking up this change will have to adapt the workflow file to keep using these actions for pre-submit testing. This has been a common request from downstream projects, but could also be done in a separate change (or not at all).

This pull request has been closed without being integrated.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1801



More information about the build-dev mailing list