-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Remove some references to JDK8 #20463
Conversation
fc9411e
to
179a94c
Compare
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building 179a94c
Full information is available in the Build summary check run. Failures⚙️ Devtools Tests - JDK 11 #- Failing: integration-tests/devtools
📦 integration-tests/devtools✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
⚙️ Devtools Tests - JDK 11 Windows #- Failing: integration-tests/devtools
📦 integration-tests/devtools✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
✖
⚙️ Gradle Tests - JDK 11 Windows #- Failing: integration-tests/gradle
📦 integration-tests/gradle✖
⚙️ JVM Tests - JDK 11 #- Failing: integration-tests/scala
📦 integration-tests/scala✖
✖
⚙️ JVM Tests - JDK 11 Windows #- Failing: integration-tests/scala
📦 integration-tests/scala✖
✖
⚙️ JVM Tests - JDK 17 #- Failing: integration-tests/scala
📦 integration-tests/scala✖
✖
|
@@ -1,30 +0,0 @@ | |||
#### | |||
# This Dockerfile is used in order to build a container that runs the Quarkus application in native (no JVM) mode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything else looks straight forward - but why deleting the two Dockerfile(s) ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had the same question when I looked at the PR and concluded that they were removed because there was no reason for them being there in the first place
179a94c
to
d1dfe3b
Compare
No description provided.