lein-figwheel

richiardiandrea 2018-06-01T17:11:58.000375Z

Hello folks, just a quick feedback on figwheel.main

richiardiandrea 2018-06-01T17:12:13.000097Z

I am trying it against my replumb and clojure -A:dev -m figwheel.main -b dev -r

richiardiandrea 2018-06-01T17:12:17.000465Z

works great

richiardiandrea 2018-06-01T17:12:21.000513Z

so 👍

richiardiandrea 2018-06-01T17:13:08.000034Z

the only small glitch was that I was trying clojure -A:dev -m figwheel.main -co dev.cljs.edn -r before, which should be equivalent...but it is not

richiardiandrea 2018-06-01T17:13:21.000389Z

becuase -co does not seem to take the main into consideration

richiardiandrea 2018-06-01T17:13:42.000400Z

clojure -A:dev -m figwheel.main -co dev.cljs.edn -c replumb.core -r is what works

richiardiandrea 2018-06-01T17:13:53.000261Z

probably an upstream issue though

bhauman 2018-06-01T17:14:00.000021Z

@richiardiandrea yeah actually -co and other opts are only for compile,build

bhauman 2018-06-01T17:14:17.000144Z

is that a mistake in my readme?

bhauman 2018-06-01T17:14:38.000015Z

I'm adding cli validation for that right now

richiardiandrea 2018-06-01T17:14:46.000585Z

trying exactly the readme command now and i will tell you

richiardiandrea 2018-06-01T17:16:56.000729Z

oh wait sorry

richiardiandrea 2018-06-01T17:17:11.000523Z

my bad.

richiardiandrea 2018-06-01T17:18:52.000020Z

confirm that these two work:

clojure -A:dev -m figwheel.main  -co dev.cljs.edn -c replumb-repl.core -r
clojure -A:dev -m figwheel.mainb dev -r
this one does not:
clojure -A:dev -m figwheel.main -co dev.cljs.edn -r

bhauman 2018-06-01T17:19:33.000441Z

yeah that make sense, thanks so much

richiardiandrea 2018-06-01T17:19:51.000074Z

np good job btw very smooth

bhauman 2018-06-01T17:19:53.000437Z

also there is a #figwheel-main

bhauman 2018-06-01T17:20:16.000267Z

I'm trying to validate the command line args to take away the guesswork

bhauman 2018-06-01T17:20:48.000535Z

so a bad command will fail early

👍 1
richiardiandrea 2018-06-01T17:20:59.000472Z

should be in core 😉

bhauman 2018-06-01T17:21:02.000532Z

and tell you why

bhauman 2018-06-01T17:21:14.000731Z

hard to argue with that

1