Skip to content

Conversation

@Janpot
Copy link
Member

@Janpot Janpot commented Oct 10, 2025

Adapting to changes for mui/mui-public#746

Note:

  • will mandate .jsx files for js files from code-infra, with only a local override for the packages folder in this repo
  • now we need extensions everywhere since eslint became a multi-format linter

@mui-bot
Copy link

mui-bot commented Oct 10, 2025

Netlify deploy preview

https://deploy-preview-47056--material-ui.netlify.app/

Bundle size report

Bundle Parsed size Gzip size
@mui/material 0B(0.00%) 0B(0.00%)
@mui/lab 0B(0.00%) 0B(0.00%)
@mui/system 0B(0.00%) 0B(0.00%)
@mui/utils 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against ee88b4d

@Janpot Janpot changed the title Json eslint [code-infra] Lint json through eslint Oct 10, 2025
@Janpot Janpot added the scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label Oct 10, 2025
@Janpot Janpot requested a review from a team October 10, 2025 12:50
@Janpot Janpot marked this pull request as ready for review October 10, 2025 14:00
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Oct 13, 2025
'react/sort-prop-types': 'off', // 228
'@typescript-eslint/ban-ts-comment': 'off', // 117
'@typescript-eslint/no-require-imports': 'off', // 133
'react/jsx-filename-extension': 'off',
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be in code-infra ?

Copy link
Member Author

@Janpot Janpot Oct 14, 2025

Choose a reason for hiding this comment

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

Yes, I'm bringing this to code infra, but separate effort. Already making sure this repo can handle the changes. other repos shouldn't be affected

@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Oct 14, 2025
@Janpot Janpot enabled auto-merge (squash) October 14, 2025 14:05
@Janpot Janpot merged commit 6a48ed1 into mui:master Oct 14, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants