Borja Lorente
Borja Lorente
@tpasternak @mai93 , would you be okay if I open a PR with the cleanups and we merge this? Some of our users really liked the feature, and it seems...
@tingilee I opened a PR with the cleanup here: https://github.com/bazelbuild/intellij/pull/6030 Please let us know in a couple of days if you'd like to incorporate the changes to yours, otherwise we'll...
@ramilmsh I've tried to reproduce the case you mention in https://github.com/blorente/intellij/blob/blorente/repro-oss-issue-with-proto-in-proto/examples/go/with_proto/proto/BUILD.bazel, and haven't been able to. Since this PR, though it may be incomplete, is useful on its own, I'd...
Closing since https://github.com/bazelbuild/intellij/pull/6030 merged.
Hello @LittleWat. While this is technically possible, it's not a workflow we currently support. Because of the way IntelliJ plugins are structured, we have to choose how we're going to...
I'm bumping this issue to P3 and removing the milestone, as it's unlikely I'll have the time to work on it this year.
@mai93 Thanks for your review! I've addressed the comments and rebased the branch.
@mai93 Thanks for the review! The issues with the tests was that the UI setup wasn't done properly for testing. Turns out, we should probably not do any UI setup...
Thanks everyone for weighing in! Added a LICENSE and CODEOWNERS file. As per the third requirement, we are still finding the time to implement it, but here is the sketch...
This work has now been upstreamed to https://github.com/bazelbuild/intellij, so closing this issue.