Thomas Broyer

Results 241 comments of Thomas Broyer

My understanding is that Eclipse is still and always built with/against gjf 1.13.0; it has never been updated: https://github.com/google/google-java-format/blob/master/eclipse_plugin/pom.xml

Well, technically, they are: Google Java Format won't change the content of the string, it cannot know whether the spaces and line ends are meaningful or not. Your IDE highlights...

This is changing the [intended](https://lit.dev/docs/frameworks/react/#why-are-wrappers-needed) and [documented](https://lit.dev/docs/frameworks/react/#how-it-works) behavior by treating everything (except events) as attributes, which: * will break if: * values aren't strings or easily convertible to/from string (converter...

@fbricon: putting annotation processors as a dependency of the plugin rather than the project is appealing but unfortunately doesn't work in all setups; see square/dagger#194. Actually, it seems like this...

FYI, I published a Gradle plugin that automatically sets up IntelliJ IDEA and/or Eclipse (in addition to adding new configurations and automatically configuring `JavaCompile` tasks): https://github.com/tbroyer/gradle-apt-plugin For IntelliJ IDEA, when...

> Follow-up question that's possibly off-topic here: does Dagger 2 have this feature? AFAICT, Dagger 2 forbids nulls by default (this can be turned to a warning globally, or turned...

[m2e-apt](https://community.jboss.org/en/tools/blog/2012/05/20/annotation-processing-support-in-m2e-or-m2e-apt-100-is-out) works fine for me, though it requires either: - explicitly putting Dagger as a dependency of the maven-compiler-plugin - explicitly listing the processors rather than relying on discovery -...

Should this issue be closed? We have #110 already (which is fixed BTW, but hasn't been closed yet) which covers error reporting as annotations in the editors, and I think...

I have to take back my above claim. Looks like an Eclipse plugin would help here; see [my comment on issue #126](https://github.com/square/dagger/issues/126#issuecomment-13550413) for more info on what's currently missing.