Skip to content

Remove vestigial useJava5 parameter - #119

Merged
elharo merged 2 commits into
masterfrom
elharo-patch-1
Apr 5, 2023
Merged

Remove vestigial useJava5 parameter#119
elharo merged 2 commits into
masterfrom
elharo-patch-1

Conversation

@elharo

@elharo elharo commented Apr 3, 2023

Copy link
Copy Markdown
Contributor

There are warnings in the build about this.

@elharo

elharo commented Apr 3, 2023

Copy link
Copy Markdown
Contributor Author

Not sure if the build is broken at head or just flaky.

@elharo

elharo commented Apr 4, 2023

Copy link
Copy Markdown
Contributor Author

Possibly Verify / Verify / ubuntu-latest jdk-11-temurin 3.9.1 (push) is broken at head. Going to try with something else.

@elharo

elharo commented Apr 4, 2023

Copy link
Copy Markdown
Contributor Author

Error: Failed to execute goal org.apache.maven.plugins:maven-pmd-plugin:3.21.0-SNAPSHOT:check (default) on project MPMD-270-325-JDK11: You have 1 PMD violation. For more details see: /home/runner/work/maven-pmd-plugin/maven-pmd-plugin/target/it/MPMD-270-325-JDK11/target/pmd.xml -> [Help 1]
Error:

This is the same error we see on #117 so I assume that test is broken there somehow

@elharo
elharo requested a review from michael-o April 5, 2023 12:03
@elharo

elharo commented Apr 5, 2023

Copy link
Copy Markdown
Contributor Author

I don't understand why some of the CI checks get cancelled. This seems a routine occurrence though. At least none are failing.

@elharo

elharo commented Apr 5, 2023

Copy link
Copy Markdown
Contributor Author

Error: The operation was canceled.

@elharo
elharo merged commit 36b92bf into master Apr 5, 2023
@elharo
elharo deleted the elharo-patch-1 branch April 5, 2023 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants