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] fix the @@
prefix issue if bazel.module
and WORKSPACE
are used together
#534
Conversation
7a11ce9
to
6818f50
Compare
Ok, so this should be much safer. I don't do any assumptions here about @ or @@. Instead I decided to allow both as both are possible within a single workspace. |
server/src/main/kotlin/org/jetbrains/bsp/bazel/server/sync/ProjectResolver.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just minor comments, but seems to work on our project
bazelrunner/src/main/kotlin/org/jetbrains/bsp/bazel/bazelrunner/BazelInfo.kt
Outdated
Show resolved
Hide resolved
bazelrunner/src/main/kotlin/org/jetbrains/bsp/bazel/bazelrunner/BazelInfo.kt
Outdated
Show resolved
Hide resolved
server/src/main/kotlin/org/jetbrains/bsp/bazel/server/sync/ProjectResolver.kt
Show resolved
Hide resolved
I confirmed it just in case on the private repository I am working on and it seems to work alright. I also rebased the PR, which seems to have made the CI issue go away. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks a lot!
bazel.module
and WORKSPACE
are used together
bazel.module
and WORKSPACE
are used together@@
prefix issue if bazel.module
and WORKSPACE
are used together
Thanks for taking a look at it! |
Second attempt, this time I changed it to looks for the actual @@ or @ label to be available.