Fork me on GitHub
#figwheel-main
<
2018-10-19
>
bhauman14:10:37

As my schedule has changed I will generally a be available in Slack on Fridays.

👍 12
dpsutton14:10:59

generally only be available on Fridays?

aisamu15:10:51

Hi! We're testing cljs with npm modules (through webpack) and ran into a small reloading woe yesterday. We have a large js codebase, so the js reloading story was marked as important++. Like the nom tutorial, there's a src/js/index.js and webpack generates /dist/index.bundle.js Using just cljs.main, (and manually setting foreign libs) we were able to pass a -w src:list to have index.bundle.js watched by the compiler. Using figwheel.main, (either setting things manually or using the extremely nice :npm option ❤️), we couldn't find a straightforward way to have the bundle watched. We expected those to be watched automatically, but when that didn't happen we tried setting up watch-dirs. Figwheel complains that there are no cljs files on the bundle folder and doesn't start. Our current workaround (no one is proud) is to add /dist/fake/core.cljs file (with a fake namespace as well). The watch option is then forwarded to the cljs compiler, and the bundle is correctly watched. Figwheel doesn't detect and reload the cljs ns's that depend on the changed foreign dependencies, but I'm not sure that's within its goals. Are we missing something?

bhauman15:10:55

yep got a job

🎉 40
dpsutton15:10:27

congrats bruce

bhauman15:10:37

@dpsutton and as you know when you start a job it can be intense at the beginning

dpsutton15:10:38

they are quite lucky

☝️ 12
bhauman15:10:44

thanks 🙂

dpsutton15:10:56

and they expect you to work for them instead of help random people on the internet

kwladyka13:10:46

half a true, you hire such people also because they bring other good people to company. Famous face in developers world 🙂

kwladyka13:10:55

but generally yeah 🙂

bhauman15:10:23

@aisamu yeah the bundle isn’t watched

aisamu15:10:28

Got it, thanks! And good luck on the new job - not that you need it!

bhauman15:10:55

thanks 🙂

bhauman15:10:24

but I should probably look at that cljs file requirement

🙂 4
4