-
Notifications
You must be signed in to change notification settings - Fork 281
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
Document IntelliJ plugin incompatibility with custom rule names #1047
Document IntelliJ plugin incompatibility with custom rule names #1047
Conversation
06b74d6
to
61ca881
Compare
61ca881
to
ab1dcb1
Compare
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.
LGTM, thanks!
One small comment inside
docs/customizable_phase.md
Outdated
|
||
Bazel IntelliJ plugin has hard-coded the names of rules_scala targets that it detects as Scala targets: | ||
|
||
https://github.com/bazelbuild/intellij/blob/master/scala/src/com/google/idea/blaze/scala/ScalaBlazeRules.java#L32-L37 |
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.
Maybe use a permalink in case it moves?
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.
Fixed.
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!
…lbuild#1047) * Document IntelliJ plugin incompatibility with custom rule names * Changing reference to intellij plugin codebase to a permalink
Description
Adding documentation about IntelliJ plugin incompatibility as requested in bazelbuild/intellij#1824.
Motivation
We need to document that custom rule names won't work with IntelliJ which is not obvious.