Skip to content

Conversation

dzbarsky
Copy link
Contributor

No description provided.

@github-actions github-actions bot added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Sep 12, 2025
@dzbarsky dzbarsky force-pushed the zbarsky/chicory branch 2 times, most recently from 42d0967 to 70f0402 Compare September 12, 2025 20:15
build --java_language_version=21
build --tool_java_language_version=21
build --tool_java_runtime_version=21
# Don't depend on a JAVA_HOME pointing at a system JDK
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe we can upgrade rules_jvm_external instead of doing this to make it unnecessary; I can do so in a separate PR

@bharadwaj08-one
Copy link

@dzbarsky Could you please take a look at the failing checks?

@bharadwaj08-one bharadwaj08-one added awaiting-user-response Awaiting a response from the author and removed awaiting-review PR is awaiting review from an assigned reviewer labels Sep 16, 2025
@meisterT
Copy link
Member

@dzbarsky perhaps just rebasing and rerunning the tests helps?

@dzbarsky
Copy link
Contributor Author

Converting to draft for now, chicory upstream is planning to make more changes to help us #26975 (comment)

@dzbarsky dzbarsky marked this pull request as draft September 19, 2025 14:39
@dzbarsky
Copy link
Contributor Author

Fixed in #27186

@dzbarsky dzbarsky closed this Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-user-response Awaiting a response from the author team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants