Skip to content

Conversation

LeFrosch
Copy link
Collaborator

@LeFrosch LeFrosch commented Sep 5, 2025

No description provided.

@LeFrosch LeFrosch marked this pull request as ready for review September 5, 2025 11:03
@github-actions github-actions bot added the awaiting-review Awaiting review from Bazel team on PRs label Sep 5, 2025
@ujohnny
Copy link
Collaborator

ujohnny commented Sep 5, 2025

let's submit feature request to renovate?

@LeFrosch
Copy link
Collaborator Author

The VirtualIncludesHandler tries to guess target names from the _virtual_includes path. This does not work with the new short_virtual_includes feature: bazelbuild/bazel#26532

Disable this feature in tests for now, we need to find a new solution with the new includes cache anyway.

@LeFrosch LeFrosch merged commit 81a3d0e into bazelbuild:master Sep 11, 2025
3 checks passed
@LeFrosch LeFrosch deleted the pull/04d3022d2392190b75337af1754c6f172d3a17f0 branch September 11, 2025 15:51
@github-project-automation github-project-automation bot moved this from Untriaged to Done in Bazel IntelliJ Plugin Sep 11, 2025
@github-actions github-actions bot removed the awaiting-review Awaiting review from Bazel team on PRs label Sep 11, 2025
sellophane pushed a commit that referenced this pull request Sep 16, 2025
NOTE: The VirtualIncludesHandler tries to guess target names from the _virtual_includes path. This does not work with the new short_virtual_includes feature: bazelbuild/bazel#26532. Disable this feature in tests for now, we need to find a new solution with the new includes cache anyway.
(cherry picked from commit 81a3d0e)
sellophane pushed a commit to sellophane/intellij that referenced this pull request Sep 18, 2025
NOTE: The VirtualIncludesHandler tries to guess target names from the _virtual_includes path. This does not work with the new short_virtual_includes feature: bazelbuild/bazel#26532. Disable this feature in tests for now, we need to find a new solution with the new includes cache anyway.
(cherry picked from commit 81a3d0e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants