material-ui icon indicating copy to clipboard operation
material-ui copied to clipboard

Published 20 hours ago • verified publisher icon mui

[core] Migrate all the internals exported by `tests/utils/index.js` to TypeScript

Open flaviendelangle opened this issue 2 years ago • 2 comments

I also removed getClasses and createShallow which were not used anywhere. If you think it is to risky, I can add them back

flaviendelangle avatar Dec 07 '22 13:12 flaviendelangle

Netlify deploy preview

No updates.

Bundle size report

No bundle size changes

Generated by :no_entry_sign: dangerJS against 3d9f5224151e935494eaa0e1049c10df008f9284

mui-bot avatar Dec 07 '22 13:12 mui-bot

I also removed getClasses and createShallow which were not used anywhere. If you think it is to risky, I can add them back

If they are not used anywhere, I am good with it. Will leave the review to @michaldudak :)

mnajdova avatar Dec 13 '22 11:12 mnajdova

@michaldudak if you can have a look

flaviendelangle avatar Feb 09 '23 10:02 flaviendelangle

Looks OK, tests pass, ship it!

michaldudak avatar Feb 09 '23 11:02 michaldudak