Skip to content

Conversation

@daddykotex
Copy link
Contributor

There!

I finally ran the tests (some weird sbt-junit-interface issue)...

Anyway the output of jvm 18 was a tidy bit different being 18.0.1

Output is now:

```
openjdk version "18.0.1" 2022-04-19
OpenJDK Runtime Environment Zulu18.30+11-CA (build 18.0.1+10)
OpenJDK 64-Bit Server VM Zulu18.30+11-CA (build 18.0.1+10, mixed mode, sharing)
```
@daddykotex
Copy link
Contributor Author

with your approval, I think this PR will be green @alexarchambault

Copy link
Member

@alexarchambault alexarchambault left a comment

Choose a reason for hiding this comment

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

Thanks @daddykotex! I only left minor comments.

daddykotex and others added 2 commits June 7, 2022 19:45
Co-authored-by: Alexandre Archambault <alexarchambault@users.noreply.github.com>
@daddykotex
Copy link
Contributor Author

should be ready to go

@daddykotex
Copy link
Contributor Author

bumping so you can trigger a build so I can any remaining issue

 Conflicts:
	interface-test/src/test/java/coursierapi/test/JvmManagerTests.java
	interface/src/main/scala/coursier/internal/api/ApiHelper.scala
Copy link
Member

@alexarchambault alexarchambault left a comment

Choose a reason for hiding this comment

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

Merging, thanks again!

@alexarchambault alexarchambault merged commit bb59b67 into coursier:master Jul 11, 2022
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.

2 participants