squint

Dag Norberg 2024-09-29T10:09:18.823039Z

I think I found a bug, could you take a look?

(ns NameCollision)

(defn Bar []
  #jsx (let [input true]
         #jsx [:input]))
This compiles to:
import * as squint_core from 'squint-cljs/core.js';
;
var Bar = function () {
{let input1 = true;
return <input1></input1>}
};

export { Bar }

borkdude 2024-09-29T10:13:21.976599Z

Ah yes, this is a bug, the tag should just be <input> right?

borkdude 2024-09-29T10:13:30.549829Z

Github issue welcome, will fix later today

Dag Norberg 2024-09-29T10:41:19.594709Z

👍 have submitted an issue

borkdude 2024-09-29T11:01:54.398019Z

0.8.114 is the latest version, you're way behind :)

borkdude 2024-09-29T11:02:59.035629Z

can't repro with the latest version

Dag Norberg 2024-09-29T11:26:44.465599Z

oh! should have thought of that 🙂 I have been using this a starting point which I can see hasn't been updated for a while: https://github.com/squint-cljs/squint/tree/main/examples/solid-js

borkdude 2024-09-29T11:35:28.434999Z

great point, I should update all of those example's versions...

borkdude 2024-09-29T12:19:46.706079Z

done

borkdude 2024-09-29T12:26:32.022829Z

btw the example also predates squint watch and squint.edn , I should also update that example to use that

Dag Norberg 2024-09-29T13:28:37.837859Z

not sure what you mean? squint.edn is currently in the repo in the solidjs directory. and squint watch as well is referenced in bb.edn

Dag Norberg 2024-09-29T13:28:52.786539Z

as of 10 months ago it says

borkdude 2024-09-29T13:31:38.612389Z

Ah all good then