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

Fix gradle command #5735

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ngocnhan-tran1996
Copy link
Contributor

Related gh-5599

I run ./gradlew :micrometer-osgi-test:check and the result is success on my local machine with OS is Windows 11.
I think we should check ./gradlew clean check before merging (maybe the result will be different)

@ngocnhan-tran1996 ngocnhan-tran1996 marked this pull request as draft December 7, 2024 10:40
@ngocnhan-tran1996 ngocnhan-tran1996 force-pushed the fix-gradle branch 3 times, most recently from c16c434 to 48f7bad Compare December 7, 2024 11:40
@jonatan-ivanov
Copy link
Member

Cannot we "just" upgrade to 7.x instead of using 6.x and 7.x simultaneously?

@shakuzen
Copy link
Member

7.x requires Java 17, I believe.

@ngocnhan-tran1996
Copy link
Contributor Author

@jonatan-ivanov

Java 11 is incompatiable with version 7.x.
It will raise error when build step run.

If just upgrade version, I think we will wait until we have a plan not support prior java 17

@shakuzen
Copy link
Member

What's remaining to move this PR to ready for review? I ran ./gradlew check on my machine (Apple M1) and it passed.

@ngocnhan-tran1996 ngocnhan-tran1996 marked this pull request as ready for review December 20, 2024 13:37
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.

3 participants