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

[core] Fix Vale errors #4347

Merged
merged 1 commit into from
Nov 2, 2024

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Nov 1, 2024

Fix errors reported by vale sync && git ls-files | grep -h -E ".(mdx|md)$" | xargs vale --filter='.Level=="error"'.

The CI should fail for them but it's not working today. @vladmoroz I have cleaned up mui/mui-public#226 to provide details on what's going on here.

I noticed this problem on #4344.

@oliviertassinari oliviertassinari added core Infrastructure work going on behind the scenes docs Improvements or additions to the documentation labels Nov 1, 2024
@oliviertassinari oliviertassinari added the scope: docs-infra Specific to the docs-infra product label Nov 1, 2024
Create a layout file for your dashboard pages (e.g., `app/(dashboard)/layout.tsx`):
Create a layout file for your dashboard pages (for example, `app/(dashboard)/layout.tsx`):
Copy link
Member Author

@oliviertassinari oliviertassinari Nov 1, 2024

Choose a reason for hiding this comment

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

The `<Account />` component is composed of multiple sub-components:
The `<Account />` component is composed of multiple subcomponents:
Copy link
Member Author

@oliviertassinari oliviertassinari Nov 1, 2024

Choose a reason for hiding this comment

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

This was reported in https://github.com/mui/toolpad/pull/4181/files

SCR-20241101-szvn

See PR description (mui/mui-public#226 (comment)) for why it didn't break the CI.

@oliviertassinari oliviertassinari merged commit dc42183 into mui:master Nov 2, 2024
16 checks passed
@oliviertassinari oliviertassinari deleted the fix-vale-errors branch November 2, 2024 11:25
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 docs Improvements or additions to the documentation scope: docs-infra Specific to the docs-infra product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants