Skip to content
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

NR-55012 Java 20 support #1226

Merged
merged 11 commits into from Apr 28, 2023
Merged

NR-55012 Java 20 support #1226

merged 11 commits into from Apr 28, 2023

Conversation

jtduffy
Copy link
Contributor

@jtduffy jtduffy commented Apr 27, 2023

  • Update ASM version
  • Update supported version regex, tests, etc
  • Update GHA workflows for java 20
  • Addition of Java20IncompatibleTest marker interface; removed Java19IncompatibleTest interface

@@ -89,7 +89,7 @@ jobs:
sed -i -e "s|jdk8=8|jdk8=${ORG_GRADLE_PROJECT_jdk8}|
s|jdk11=11|jdk11=${ORG_GRADLE_PROJECT_jdk11}|
s|jdk17=17|jdk17=${ORG_GRADLE_PROJECT_jdk17}|
s|jdk19=19|jdk19=${ORG_GRADLE_PROJECT_jdk19}|" gradle.properties.gha
s|jdk20=20|jdk20=${ORG_GRADLE_PROJECT_jdk20}|" gradle.properties.gha
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few places that we set a envvar GRADLE_PROPERTIES, and the ORG_GRADLE_PROJECT_jdk19 should be changed there also.

@jtduffy jtduffy merged commit a083ca3 into main Apr 28, 2023
366 of 405 checks passed
@jtduffy jtduffy deleted the NR-55012-J20-support branch April 28, 2023 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants