-
Notifications
You must be signed in to change notification settings - Fork 864
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
google-java-format Eclipse plugin does not provide AOSP Style option as google-java-format IntelliJ plugin #179
Comments
The easiest way is to install a second "Formatter implementation" plugin that provides AOSP style formatting out of the box. Then the user may choose the first or second implementation in the drop-down box shown in your screenshot. |
This is the line that creates the Formatter with default options: https://github.com/google/google-java-format/blob/master/core/src/main/java/com/google/googlejavaformat/java/SnippetFormatter.java#L60 |
Thank you! @sormuras |
@sormuras |
AFAIK, there's no (easy) possibility to present custom formatter options for the selected formatter implementation in Eclipse. The Eclipse plugin contains an entire GJF distro -- using a default This is what I'd do:
Finally, use https://jitpack.io to build and host your fork with two eclipse plugins. |
I've create a PR to address this: #251 |
It is weird a solution to this issue doesn't seem to be needed. Looks like nobody is using Eclipse anymore and I'm the only one. After playing around a little bit with |
I still would like to use the AOSP style with eclipse and I would greatly appreciate any kind of feedback from Google about the PR #251 The XML files for Eclipse do not behave the same as the actual formatter implementation. I see them more as an approximation of the real thing. I actually think that the XML files should be removed. See google/styleguide#687 |
The google-java-format Eclipse plugin works in Eclipse IDE for Java Developers
Version: Oxygen Release (4.7.0)
Build id: 20170620-1800
But it does not provide AOSP Style option. Is this intentional design?
Most members of my team like AOSP style, while half of them are using Eclipse.
Is there a workaround to get over this problem. E.g. can you expose an XML configuration file for AOSP.
Thank you!
The text was updated successfully, but these errors were encountered: