[core] Update supported Node.js version to 14

[Janpot]

It looks like these were forgotten in #42920? This would allow us to add package "exports" on a minor.

[aarongarciah]

Nice catch. Maybe we can update @mui/utils too?

material-ui/packages/mui-utils/package.json

Line 79 in 2f7cc7b

"node": ">=12.0.0"

edit: nevermind, you already pushed a commit addressing it 🦸‍♂️

[mui-bot]

Netlify deploy preview

• docs/data/material/getting-started/supported-platforms/supported-platforms.md

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 8434ecf

[aarongarciah]

Thanks Jan!

[Janpot]

@aarongarciah Making a few changes to fix the errors introduced in #43290 and prevent them in the future.

cc @JCQuintas this is why I push back against running ci conditionally. So easy to break, especially when it's manually maintained.

edit: removing the fix again, it's handled by #43316

[JCQuintas]

@aarongarciah Making a few changes to fix the errors introduced in #43290 and prevent them in the future.

cc @JCQuintas this is why I push back against running ci conditionally. So easy to break, especially when it's manually maintained.

edit: removing the fix again, it's handled by #43316

I don't see where the issue was with the CI itself? I guess it was a typescript issue?

I don't think it is inherently bad. If we always run all the CI we can get cost issues or time issues. If we optimise for running only in very specific scenarios we run the chance of making them brittle.

Each case is a different case. So we should always take them into account when making decisions.