New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[material] Remove unused dev dependency on @mui/lab #41198
Conversation
brijeshb42
commented
Feb 20, 2024
- I have followed (at least) the PR section of the contributing guide.
Netlify deploy previewhttps://deploy-preview-41198--material-ui.netlify.app/ Bundle size report |
This looked like a circular dependency 😁 |
For some reason there is a test in @mui/material that imports from @mui/lab: packages/mui-material/test/typescript/color-palette-prop.spec.tsx. I think the part of this test that references TimelineDot can be moved to the Lab.
Yeah, it wasn't reported by pnpm, though, as @mui/material is a peer dependency. |
2b18267
to
5cef954
Compare
5cef954
to
bafa5c3
Compare
It was always reported when I tried to run some command directly through |