-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[core] Add @mui/system
to install docs to avoid missing peer dependency
#14832
Comments
Hello, could you clarify the use case you are referring to? 🤔 Please provide a minimal reproduction test case with the latest version. This would help a lot 👷. |
I can create a minimal repro, but it's simply adding the dependencies and seeing the above warning. Is what #11128 says about moving to explicitly installing @mui/system no longer valid? |
@mui/system
to install docs to avoid missing peer dependency
It is no longer valid because there was a recent addition to the |
So if I go to Stackblitz to make a minimal repro, and just update the package.json to look like this:
|
Is there something I can do to follow this up? The above is the best repro I can offer. You see the issue when trying to even set up a repro on Stackblitz. |
@markedwards Stackblitz doesn't support optional peer dependencies. 🙈 In regards to your peer dependency message, it seems setup dependent. Does it answer your question? 🤔 |
Perhaps a dumb question, but would it make sense to just have In any case, yes my question is definitely answered, thanks. |
No questions are dumb. 🙈 |
Yeah, I figured that was the reasoning, but I wanted to confirm. I think I will just remove the |
This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue. Note @markedwards How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey. |
@mui/system
to install docs to avoid missing peer dependency@mui/system
to install docs to avoid missing peer dependency
Summary
According to #11128,
@mui/system
is expected to be explicitly installed. However, it seems to have never been added to the installation instructions.Is it expected to be installed? If not, why the missing peer dependency warning?
Examples
Motivation
No response
Search keywords: @mui/system
The text was updated successfully, but these errors were encountered: