ylemkimon

Results 60 comments of ylemkimon

- ~`\ifinner`~ - `\iftrue` - `\iffalse`

I'm against adding `\htmlAttr` as (1) validation of attributes is impossible and (2) it may open to vulnerabilities and blacklisting is not enough.

@junghoocho That seems to be OK. If you don't mind, could you update the PR? I'll leave #2265 open for discussions on allowing other attributes.

The failing `Screenshotter / screenshotter (safari)` is the old screenshotter (Browserstack) and the `CI / screenshotter (safari)` is the new screenshotter (macOS runner).

> With the current setup, I am able to generate Safari screenshots locally (by adding my Browserstack credentials to the environment), but relying on CI would prevent this (unless I...

`screenshotter.sh` doesn't support specifying the browser, but yes, it'd be useful.

This incorporates changes from #2085, so it'd be preferable to review it first.

Will be fixed by #2139. No timeline yet, sorry.

@Djay77 see https://github.com/npm/cli/issues/2660.