Skip to content

Upgrade rules_java and remove experimental_google_legacy_api flag#18889

Merged
zhangskz merged 1 commit intoprotocolbuffers:29.xfrom
comius:cherry-pick-rules_java_upgrade
Oct 18, 2024
Merged

Upgrade rules_java and remove experimental_google_legacy_api flag#18889
zhangskz merged 1 commit intoprotocolbuffers:29.xfrom
comius:cherry-pick-rules_java_upgrade

Conversation

@comius
Copy link
Contributor

@comius comius commented Oct 17, 2024

The upgrade of rules_java also upgrades error prone and this resulted in some additional errors reported in Protobuf's tests. Set the flag to report those as warning, but they should be cleaned up eventually.

PiperOrigin-RevId: 686565803

@comius comius requested review from a team as code owners October 17, 2024 21:07
@comius comius requested review from Logofile and mkruskal-google and removed request for a team October 17, 2024 21:07
@mkruskal-google mkruskal-google added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 17, 2024
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 17, 2024
The upgrade of rules_java also upgrades error prone and this resulted in some additional errors reported in Protobuf's tests. Set the flag to report those as warning, but they should be cleaned up eventually.

PiperOrigin-RevId: 687070204
@comius comius force-pushed the cherry-pick-rules_java_upgrade branch from 1e70812 to 7cf2df5 Compare October 17, 2024 22:57
@zhangskz zhangskz added the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 18, 2024
@zhangskz zhangskz enabled auto-merge (squash) October 18, 2024 14:46
@github-actions github-actions bot removed the 🅰️ safe for tests Mark a commit as safe to run presubmits over label Oct 18, 2024
@zhangskz zhangskz merged commit f25389b into protocolbuffers:29.x Oct 18, 2024
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.

4 participants