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] Rename "docs-base" path alias to "docs" #742

Merged
merged 4 commits into from
Oct 17, 2024

Conversation

vladmoroz
Copy link
Contributor

No description provided.

@michaldudak michaldudak added the core Infrastructure work going on behind the scenes label Oct 17, 2024
@mui-bot
Copy link

mui-bot commented Oct 17, 2024

Netlify deploy preview

https://deploy-preview-742--base-ui.netlify.app/

Generated by 🚫 dangerJS against f0663f4

michaldudak
michaldudak previously approved these changes Oct 17, 2024
@michaldudak michaldudak dismissed their stale review October 17, 2024 14:16

Failing scripts

@@ -88,7 +88,9 @@ async function transpileFile(tsxPath, project) {
if (enableJSXPreview) {
transformOptions.plugins = transformOptions.plugins.concat([
[
require.resolve('docs/src/modules/utils/babel-plugin-jsx-preview'),
require.resolve(
'node_modules/@mui/monorepo/docs/src/modules/utils/babel-plugin-jsx-preview',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should work as well:

Suggested change
'node_modules/@mui/monorepo/docs/src/modules/utils/babel-plugin-jsx-preview',
'@mui/monorepo/docs/src/modules/utils/babel-plugin-jsx-preview',

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep that works, thanks!

Comment on lines 91 to 93
require.resolve('docs/src/modules/utils/babel-plugin-jsx-preview'),
require.resolve(
'node_modules/@mui/monorepo/docs/src/modules/utils/babel-plugin-jsx-preview',
),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this was the only place where the monorepo docs package was referenced
I am not even sure we want to display the JS demos, but for now I think this is fine

@vladmoroz vladmoroz enabled auto-merge (squash) October 17, 2024 14:30
@vladmoroz vladmoroz merged commit 50c2458 into mui:master Oct 17, 2024
18 checks passed
@vladmoroz vladmoroz deleted the docs-base branch October 17, 2024 15:58
@vladmoroz vladmoroz changed the title Rename "docs-base" path alias to "docs" [docs] Rename "docs-base" path alias to "docs" Oct 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants