Mateusz Mazurczak
Mateusz Mazurczak
Why no one is merging this :(
@liquidz it was because used through babashka, but you've fixed it already
@kkinnear Thank you very much for your answer! I've used ns-justify and I actually spent few hours before asking just reading all the options, trying to create something with fn-map...
@kkinnear like it should be: (:require [automaton-web.configuration.protocol :as web-prot] [automaton-core.configuration.protocol :as core-conf-prot] [automaton-web.configuration.files :as web-conf-files]) -> (:require [automaton-core.configuration.protocol :as core-conf-prot] [automaton-web.configuration.files :as web-conf-files] [automaton-web.configuration.protocol :as web-prot]) so automaton-core is first...
Btw our config is this for now: ``` {:style [:community :how-to-ns :require-justify :sort-dependencies :backtranslate :hiccup] :map {:comma? false :force-nl? true} :pair {:force-nl? true} :pair-fn {:hang? false} :comment {:wrap? false}} ```
@kkinnear If you need some clarification or some testing I'm here :)
Thank you for the great work @kkinnear! Feature wise this is everything we need. Nice that you thought about having many regexes! I agree reader conditionals should not be moved,...
@kkinnear can we test those changes on some branch? :D
@kkinnear I'm okay with waiting two weeks, I don't want to put any pressure on you. It's just that you said before that you need my engagement for you to...
Sorry for a late response, I had some small problems, with using that binary on mac, but it's all good now. (I had to do chmod +x on the file...