core.edn doesn't update when I export new vars in core.js

[lilactown]

Editing core.js, I cannot convince bb dev or bb build to use the core.js file on disk to update core.edn.

[borkdude]

Try wiping the .shadow-cljs directory first

[lilactown]

I "fixed" it by copying the core.js file to node_modules/squint-cljs/core.js, since that's where the bb tasks look to get the exports.

Seems like it ought to look at ./core.js instead. I can open a PR if that's the case.

[borkdude]

Hmm yeah, makes sense

…

On Mon, Feb 20, 2023 at 9:52 PM Will Acton ***@***.***> wrote: I "fixed" it by copying the core.js file to node_modules/squint-cljs/core.js, since that's where the bb tasks look to get the exports. Seems like it ought to look at ./core.js instead. I can open a PR if that's the case. — Reply to this email directly, view it on GitHub <#294 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACFSBVFVCLYTJVI2T2KD3LWYPKSPANCNFSM6AAAAAAVCFIDBA> . You are receiving this because you commented.Message ID: ***@***.***>

-- https://www.michielborkent.nl https://www.eetvoorjeleven.nu

[borkdude]

I think you just need to call npm install and then it will create a symlink from node_modules to the current directory:

 $ ls -la node_modules/squint-cljs
lrwxr-xr-x  1 borkdude  staff  2 Dec 27 17:25 node_modules/squint-cljs -> ..

This will allow you to make changes locally in core.js and immediately use them in tests.