cljs-dev

dnolen 2025-04-16T10:55:27.417359Z

@thheller re: bumping the compiler - didn't you get a bunch of warnings from Google Closure while compiling about this removed Closure directive?

dnolen 2025-04-16T10:55:29.761559Z

/goog/dom/tagname.js:45:17: WARNING - [JSC_PARSE_ERROR] Parse error. unknown @suppress parameter: unusedPrivateMembers
  45|   /** @suppress {unusedPrivateMembers} */

dnolen 2025-04-16T10:55:31.786079Z

etc.

dnolen 2025-04-16T10:57:15.486129Z

Wondering if we shouldn't first remove those from GCL and cut a release of that as well.

thheller 2025-04-16T10:59:48.706539Z

I disabled those warnings years ago 😛

dnolen 2025-04-16T11:10:46.754679Z

@thheller oh the unknown parameter warning flag?

thheller 2025-04-16T11:11:10.469019Z

all of them

dnolen 2025-04-16T11:11:33.130259Z

hrm, ok - isn't that annoying if you're writing JS or can you re-enable them manually?

thheller 2025-04-16T11:13:16.298989Z

cljs mostly doesn't use any annotations. closure I assume is "fine". npm often has annotations where they mean completely different stuff than closure, so was causing errors all over the place

thheller 2025-04-16T11:15:27.355879Z

hmm actually no. I only turned them off completely for npm code. cljs/closure still runs with them on

dnolen 2025-04-16T11:15:41.095769Z

ok - yeah that would be extreme 🙂

dnolen 2025-04-16T11:16:49.483359Z

so you are putting NPM code through Closure - but is that as separate pass or something where you disable a bunch of things? or did you figure out a way to make it work more consistently?

thheller 2025-04-16T11:18:03.220789Z

separate :simple path yes. basically just collects all used npm sources, runs them through :simple together and prepends that to the other output.

dnolen 2025-04-16T11:18:59.559189Z

ok that makes sense

dnolen 2025-04-16T11:19:36.135209Z

ok cool, I think these warnings will be super annoying so I'd like to modify GCL and cut a release of our own first

thheller 2025-04-16T11:19:53.405079Z

yeah makes sense

thheller 2025-04-16T11:23:49.158039Z

good thing you have a test suite for this. I guess I didn't run into this because none of my code uses goog.dom stuff