javascript with calls to String.raw gets incorrectly minified

[perrin4869]

I put together a small reproduction here

[tdewolff]

Thanks Julian, should be fixed now!

[perrin4869]

thanks for the lightning speed fix 😄
when you have time, can you update the npm module? :)

[tdewolff]

Just churned out a new release, the NPM release should come up shortly after the GitHub action finishes.

[perrin4869]

seems the CI failed because the macos image doesn't have golang installed?

[tdewolff]

This is now fixed in the latest release!

[perrin4869]

Yeah, I was trying it earlier, thank so much!!
Only thing is, I am running into a problem where the fix doesn't seem to be applied in my CI, and I think it is because it is running arm Linux, the fix is working perfectly on my x86 Linux desktop... any clues?
Edit: this is related to the node binding

[tdewolff]

Are you sure it is downloading the latest commit from GitHub? Maybe add a go get -u github.com/tdewolff/minify/v2@v2.20.24 or @latest to make sure it uses the latest version (I've noticed it does some caching sometimes).

Otherwise, I believe that x86 will use the precompiled binaries from the GitHub workflow, but ARM will probably need to build from source, but that should just work...not sure what the problem could be...

[perrin4869]

maybe it is because this is not updated?

minify/bindings/js/go.mod

Line 6 in a32f60e

github.com/tdewolff/minify/v2 v2.12.5

Since there are no prebuilds for arm on linux, it would be building the bindings from here, right?

[perrin4869]

btw, it seems we can soon run arm linux github actions :)
https://github.blog/changelog/2023-10-30-accelerate-your-ci-cd-with-arm-based-hosted-runners-in-github-actions/

[tdewolff]

You're right, I've made some changes, do they look good to you? If so, I'll release a
new version for you to test.

PS: ARM support would be a great relieve!

[perrin4869]

I don't think those would work, because the makefile is calling git to produce the hash, but the node module isn't a git repository... I think you'd need to get those values and inject them into the make file at build time