Skip to content
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

[docs] Fix webpack file name to the standard: webpack.config.js #34446

Merged
merged 1 commit into from Sep 24, 2022

Conversation

CodingItWrong
Copy link
Contributor

@CodingItWrong CodingItWrong commented Sep 23, 2022

In the Styled Engine Guide, there is an example of configuring an alias in webpack. The file shown is labeled webpack.alias.js; this PR updates it to webpack.config.js. Preview link

I think the intent here is just to indicate "put this in your webpack config," and webpack.config.js is the default webpack config file name. Additionally, the webpack docs for resolve.alias show the configuration in webpack.config.js. Also, in trying out this config option I added it to my webpack.config.js and it succeeded.

There are a few references around the web to webpack.alias.js, but none in the webpack docs. I believe what happened here is that was the author was thinking about the alias configuration, and accidentally put that into the file name.

@mui-bot
Copy link

mui-bot commented Sep 23, 2022

No bundle size changes

Generated by 🚫 dangerJS against f541353

@ZeeshanTamboli ZeeshanTamboli added docs Improvements or additions to the documentation package: styled-engine Specific to @mui/styled-engine labels Sep 24, 2022
@ZeeshanTamboli ZeeshanTamboli changed the title Update webpack file name to webpack.config.js [docs] Update webpack file name to webpack.config.js Sep 24, 2022
@oliviertassinari oliviertassinari changed the title [docs] Update webpack file name to webpack.config.js [docs] Fix webpack file name to the standard: webpack.config.js Sep 24, 2022
@oliviertassinari oliviertassinari merged commit 2890881 into mui:master Sep 24, 2022
@oliviertassinari
Copy link
Member

@CodingItWrong It's a great first pull request on MUI 👌🏻. Thank you for working on it!

alexfauquette pushed a commit to alexfauquette/material-ui that referenced this pull request Oct 14, 2022
daniel-rabe pushed a commit to daniel-rabe/material-ui that referenced this pull request Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation package: styled-engine Specific to @mui/styled-engine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants