Skip to content

Conversation

JohnTortugo
Copy link
Contributor

@JohnTortugo JohnTortugo commented Apr 16, 2025

Work in Progress


Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue

Integration blockers

 ⚠️ The commit message does not reference any issue. To add an issue reference to this PR, edit the title to be of the format issue number: message. (failed with updated jcheck configuration in pull request)
 ⚠️ Too few reviewers with at least role reviewer found (have 0, need at least 1) (failed with updated jcheck configuration in pull request)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/24702/head:pull/24702
$ git checkout pull/24702

Update a local copy of the PR:
$ git checkout pull/24702
$ git pull https://git.openjdk.org/jdk.git pull/24702/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 24702

View PR using the GUI difftool:
$ git pr show -t 24702

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/24702.diff

@bridgekeeper
Copy link

bridgekeeper bot commented Apr 16, 2025

👋 Welcome back cslucas! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk
Copy link

openjdk bot commented Apr 16, 2025

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@openjdk
Copy link

openjdk bot commented Apr 16, 2025

@JohnTortugo The following labels will be automatically applied to this pull request:

  • graal
  • hotspot

When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing lists. If you would like to change these labels, use the /label pull request command.

@openjdk
Copy link

openjdk bot commented May 13, 2025

@JohnTortugo this pull request can not be integrated into master due to one or more merge conflicts. To resolve these merge conflicts and update this pull request you can run the following commands in the local repository for your personal fork:

git checkout jvmci-installed-code-reason
git fetch https://git.openjdk.org/jdk.git master
git merge FETCH_HEAD
# resolve conflicts and follow the instructions given by git merge
git commit -m "Merge master"
git push

@openjdk openjdk bot added the merge-conflict Pull request has merge conflict with target branch label May 13, 2025
@JohnTortugo JohnTortugo force-pushed the jvmci-installed-code-reason branch from 12d54ee to b7dc33b Compare June 5, 2025 18:42
@openjdk openjdk bot removed the merge-conflict Pull request has merge conflict with target branch label Jun 5, 2025
@JohnTortugo JohnTortugo force-pushed the jvmci-installed-code-reason branch 5 times, most recently from 8212017 to 9f79d66 Compare June 9, 2025 19:05
JohnTortugo and others added 15 commits June 9, 2025 19:07
… move mouse off screen

Reviewed-by: honkar, prr
Co-authored-by: Alan Bateman <[email protected]>
Co-authored-by: Alex Menkov <[email protected]>
Reviewed-by: sspitsyn, kevinw
…and paragraph separator show non-zero advance

8356812: Create an automated version of TextLayout/TestControls

Reviewed-by: prr, honkar
…ssDefFoundError: jtreg/SkippedException

Reviewed-by: joehw, jlu, iris
…heap-buffer-overflow in ArchiveBuilder

Reviewed-by: shade, iklam, asmehra
Ian Graves and others added 27 commits June 9, 2025 19:08
…- expected [128] but found [256]

Co-authored-by: Paul Sandoz <[email protected]>
Co-authored-by: Jaikiran Pai <[email protected]>
Reviewed-by: syan, psandoz
Reviewed-by: mgronlun, mdoerr, pchilanomate, apangin, shade
…ZIP file system's documentation

Reviewed-by: dfuchs, vyazici, alanb, lancea
…ames as part of JVMTI PopFrame

Reviewed-by: dholmes, eosterlund
Reviewed-by: rgiulietti, vyazici
… not report on missing man page files

Reviewed-by: hannesw
…va does not build after JDK-8351594

Reviewed-by: alanb
…THREAD_FACTORY=Virtual

Reviewed-by: lmesnik, dholmes, sspitsyn, syan
8355748: Add SourceVersion.RELEASE_26
8355751: Add source 26 and target 26 to javac

Co-authored-by: Joe Darcy <[email protected]>
Reviewed-by: iris, coleenp, darcy
Reviewed-by: erikj, ihse, iris
@JohnTortugo JohnTortugo force-pushed the jvmci-installed-code-reason branch from 9f79d66 to 7d814d2 Compare June 9, 2025 19:13
@openjdk openjdk bot added the merge-conflict Pull request has merge conflict with target branch label Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

graal [email protected] hotspot [email protected] merge-conflict Pull request has merge conflict with target branch

Development

Successfully merging this pull request may close these issues.