You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Nick Dimiduk (Jira)" <ji...@apache.org> on 2019/11/18 18:36:00 UTC
[jira] [Commented] (HBASE-23234) Checkstyle and formatter disagree
on indentation
[ https://issues.apache.org/jira/browse/HBASE-23234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16976774#comment-16976774 ]
Nick Dimiduk commented on HBASE-23234:
--------------------------------------
Looks like there's now a lingua franca for editor code formatting configs, [Editor Config|https://editorconfig.org/]. IntelliJ supports defining a configuration from a checkstyle config, and will export its configuration in this editor config format.
> Checkstyle and formatter disagree on indentation
> ------------------------------------------------
>
> Key: HBASE-23234
> URL: https://issues.apache.org/jira/browse/HBASE-23234
> Project: HBase
> Issue Type: Task
> Components: build, tooling
> Reporter: Nick Dimiduk
> Priority: Major
>
> I don't have an exhaustive analysis of the issue, but there's at least one case where check style plugin configuration disagrees with our settings in {{dev-support/hbase_eclipse_formatter.xml}}.
> Formatter settings produce this code chunk
> {noformat}
> uncaughtExceptionHandler =
> (t, e) -> abort("Uncaught exception in executorService thread " + t.getName(), e);
> {noformat}
> but check style wants
> {noformat}
> uncaughtExceptionHandler =
> (t, e) -> abort("Uncaught exception in executorService thread " + t.getName(), e);
> {noformat}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)