CI failing with op_mob browserslist error

[cmorten]

• @testing-library/dom version: latest
• Testing Framework and version: N/A
• DOM Environment: N/A

Relevant code or config:

N/A

What you did:

Run npm run setup -s locally or in CI.

What happened:

$ kcd-scripts build  --no-ts-defs --ignore "**/__tests__/**,**/__node_tests__/**,**/__mocks__/**" && kcd-scripts build --no-ts-defs --bundle --no-clean
Error [BrowserslistError]: Unknown version 64 of op_mob

Reproduction:

npm run setup -s

Problem description:

op_mob 64 is not supported, as noted in #1170 (comment).

Suggested solution:

Move to use op_mob 73 provided it doesn’t result in a breaking change, otherwise a different config to unblock build.

[MatanBobi]

Hi @cmorten :)
I already pushed a fix to your PR for this one, I saw this before and didn't get a chance to fix it.

[eps1lon]

Please fix this in a different PR so that we can better bisect/revert later.

[MatanBobi]

I'm re-opening this because what we have at the moment is a workaround (we've pinned the version) :)

[MatanBobi]

This issue is more urgent than we thought because our workaround fails our node 14 CI because npm 6 doesn't support overrides.

[nickserv]

So #1255 should fix this, right?

[MatanBobi]

So #1255 should fix this, right?

It should fix the failure but it's still a workaround. We should probably update the browsers we support at some point I guess.

[nickserv]

That's just a matter of updating browserslist, right? Since we're doing a major anyway, let's include a fix then.

[MatanBobi]

That's just a matter of updating browserslist, right? Since we're doing a major anyway, let's include a fix then.

Yes, that's a really good point. I'll do it and create a PR to the alpha branch.