Eric Ferreira
Eric Ferreira
> Hey @ericbf Thanks for this PR - the type improvements and validation additions are definitely valuable improvements to the `env` helper. 👍 > > I wanted to discuss two...
@Bassel17 I’ve reduced the scope of this PR to just the type fixes. Can you re-open? Thanks! Or let me know and I can open a new PR with the...
I’ve also created a new PR with the same changes (#24881). We can close that one if this one gets re-opened. But I don’t want this to get missed as...
Can we re-open this?
This is still an issue on the latest version webfont, `3.34.1`: This is an issue in chrome, seems to work in Safari.
What’s the hold-up here?
@fratzinger can you resolve the merge conflicts?
For example, with this Dockerfile: ```dockerfile #syntax=docker/dockerfile:1 FROM docker.io/node:22-bookworm RUN npm install -g pnpm WORKDIR /app RUN pnpm install [email protected] CMD ls "node_modules/muhammara" ``` You can see that the `binding`...
Ok, I see. Looks like `pnpm@10` no longer runs install scripts automatically. ``` # pnpm i --loglevel verbose [email protected] Packages: +13 +++++++++++++ Progress: resolved 13, reused 13, downloaded 0, added...
This is probably a documentation change in MuhammaraJS then, if anything.