-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: add IntelliJ IDEA plugin for generating with bzlgen #48
base: master
Are you sure you want to change the base?
Conversation
TODO:
Currently bzlgen needs to be installed globally for this to work, or on the PATH somewhere, I wonder if we can rollup bzlgen and bundle it in the jar, at least then the plugin and binary are versioned in step |
if (selected.isDirectory()) { | ||
selected.refresh(false, false); | ||
} else { | ||
selected.getParent().refresh(false, false); | ||
} |
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.
I'm not convinced this is doing what I think it's doing
name = "intellij_ce_2019_3", | ||
build_file = "@//third_party/intellij_sdk:BUILD.idea193", | ||
sha256 = "fb347c3c681328d11e87846950e8c5af6ac2c8d6a7e56946d3a10e6121d322f9", | ||
url = "https://www.jetbrains.com/intellij-repository/releases/com/jetbrains/intellij/idea/ideaIC/2019.3.2/ideaIC-2019.3.2.zip", |
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.
Making sure people have the right version of intellij could be fun
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.
Yah I don't really know the compatibility here, I just went with the version I have installed now. It's probably worth checking. These deps are set to neverlink
so they don't make it into the final deploy jar, so just depends if the IJ API has changed I guess
Adds OSS version of bzlgen IntelliJ IDEA plugin for bzlgen
closes #47