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.

Sign up for GitHub

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

Jump to bottom

Update to Bazel 7, Kotlin 1.9, and JDK 21 #879

Closed
wants to merge 4 commits into from
Closed

Update to Bazel 7, Kotlin 1.9, and JDK 21 #879

wants to merge 4 commits into from

Conversation

fmeum
Copy link
Contributor

@fmeum fmeum commented Oct 30, 2023

No description provided.

@ci-fuzz-check
Copy link

ci-fuzz-check bot commented Oct 30, 2023

findings-found-icon 4 open findings detected  View Findings
The final number may be higher than displayed as the scan progresses in the background. For more information, check the Code Intelligence Documentation

@ci-fuzz-check
Copy link

ci-fuzz-check bot commented Oct 30, 2023

findings-found-icon 2 open findings detected  View Findings
The final number may be higher than displayed as the scan progresses in the background. For more information, check the Code Intelligence Documentation

@fmeum fmeum marked this pull request as ready for review November 2, 2023 13:20
@fmeum fmeum requested a review from a team November 2, 2023 13:20
@fmeum fmeum changed the title Update to Bazel 7 Update to Bazel 7, Kotlin 1.9, and JDK 21 Nov 2, 2023
fmeum added 4 commits November 6, 2023 17:53
@fmeum
deps: Update Kotlin to 1.9
5d8eda1 
This also requires updating rules_kotlin, which in turn requires some
changes to target names.
@fmeum
bazel: Update Bazel to 7.0.0rc2
e111e39 
Remove flags that are now enabled by default and silence a javac warning
caused by compiling for Java 8 on JDK 21.
@fmeum
bazel: Silence more protobuf warnings
94c8707
@fmeum
bazel: Run tests on JDK 21
58c2173 
Locally, we compile against and run tests on JDK 21, which gives us
the best performance as well as the ability to (conditionally) use newer
APIs without having to resort to reflection.
@fmeum fmeum closed this Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fmeum