vvvvalvalval / scope-capture

Project your Clojure(Script) REPL into the same context as your code when it ran
MIT License
573 stars 14 forks source link

code format & simplify some macro #2

Closed jiacai2050 closed 6 years ago

vvvvalvalval commented 6 years ago

Hi Jiacai, I appreciate and am thankful of your willingness to improve the library, however I find that the proposed changes bring mostly more concision at the expense of readability, a tradeoff which I don't find beneficial in this case. In particular, the use of explicit parameter names and certain indentation strategies were by design for readability. Granted, this is largely a matter of personal taste, but being the lead maintainer I'd rather we stick to my subjective preferences for these choices, especially when they don't violate any Clojure convention for code style.

So I'm closing the PR now, thanks again and I hope this won't deter you from offering further contributions :) don't hesitate to reach out to me for discussion before investing your time in code.

jiacai2050 commented 6 years ago

I'm fine with your code style. However I think placing all the parentheses at the end of the function is preferable among Lisp community.