Unexpected result: Comments moved outside of form

[jeaye]

Input source

(def default-data-readers
  (merge
    ; {'uuid #'clojure.uuid/default-uuid-reader}
    ; (when-class "java.sql.Timestamp"
    ;   {'inst #'clojure.instant/read-instant-date})
    ))

Command

❯ standard-clj --version
v0.20.0

❯ standard-clj fix tmp.clj

Output

(def default-data-readers
  (merge))
    ; {'uuid #'clojure.uuid/default-uuid-reader}
    ; (when-class "java.sql.Timestamp"
    ;   {'inst #'clojure.instant/read-instant-date})

Thoughts

This really intrudes with interactive development, especially if we're formatting on save. If I temporarily want to comment out the args to a fn (forget merge and imagine it's anything), standard-clj is going to put the comment after the fn call instead.

If this is what you want to happen here, fair enough. I think it'll be a hassle, though. Perhaps that comes from me being ok with ) on its own line.

Use case

This came straight outta formatting jank's clojure.core. After running fix on it, I've been scanning the diff to see what standard-clj has changed.

[oakmac]

See Issue #60 and Issue #155 for a pretty thorough discussion of this behavior.

The tldr from this comment:

Standard Clojure Style will always gather trailing parens to a single line (as recommended by the Clojure Style Guide and enforced by Parinfer). If you want a ; comment line to 'hold' some parens on the next line, place a comma before them to explicitly show that intent.

[jeaye]

Gah, I should've looked. Thanks, Chris! I'll close this one for ya.