Skip to content
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

Conversation

gergelyfabian
Copy link
Contributor

@gergelyfabian gergelyfabian commented May 20, 2020

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.

@gergelyfabian gergelyfabian force-pushed the document_intellij_plugin_hard_coded_targets branch from 61ca881 to ab1dcb1 Compare May 20, 2020 11:47
Copy link
Member

@ittaiz ittaiz left a 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


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
Copy link
Member

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

Copy link
Member

@ittaiz ittaiz left a comment

Choose a reason for hiding this comment

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

Thanks!

@ittaiz ittaiz merged commit 3dd2e9f into bazelbuild:master May 20, 2020
@gergelyfabian gergelyfabian deleted the document_intellij_plugin_hard_coded_targets branch May 21, 2020 05:14
SocksDevil pushed a commit to SocksDevil/rules_scala that referenced this pull request Jul 6, 2020
…lbuild#1047)

* Document IntelliJ plugin incompatibility with custom rule names

* Changing reference to intellij plugin codebase to a permalink
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants