New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move PR target ref resolution to the utilities class #940
Conversation
|
Webrevs
|
@rwestberg This change now passes all automated pre-integration checks. After integration, the commit message for the final commit will be:
You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed. At the time when this comment was updated there had been 1 new commit pushed to the
Please see this link for an up-to-date comparison between the source branch of this pull request and the
|
/integrate |
@rwestberg Since your change was applied there has been 1 commit pushed to the
Your commit was automatically rebased without conflicts. Pushed as commit c23ff3b. |
The target ref resolution is better done locally to avoid races and also unnecessary API calls when only the ref name is used.
Progress
Testing
Reviewers
Download
$ git fetch https://git.openjdk.java.net/skara pull/940/head:pull/940
$ git checkout pull/940