Skip to content

Comments

[code-infra] Enable undefined addition to optional properties#47750

Merged
brijeshb42 merged 3 commits intomui:masterfrom
brijeshb42:opt-undefined
Feb 19, 2026
Merged

[code-infra] Enable undefined addition to optional properties#47750
brijeshb42 merged 3 commits intomui:masterfrom
brijeshb42:opt-undefined

Conversation

@brijeshb42
Copy link
Contributor

This is done via an eslint autofix. Fixes #28745

@brijeshb42 brijeshb42 requested a review from a team February 11, 2026 18:01
@brijeshb42 brijeshb42 added the scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label Feb 11, 2026
@mui-bot
Copy link

mui-bot commented Feb 11, 2026

Netlify deploy preview

https://deploy-preview-47750--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 e5c7a7d

@brijeshb42 brijeshb42 force-pushed the opt-undefined branch 2 times, most recently from d89b5b2 to b48f4d3 Compare February 12, 2026 08:57
Copy link
Member

@Janpot Janpot left a comment

Choose a reason for hiding this comment

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

I skimmed about 15% of the changes, looks ok to me.

Some nitpicks about eslint config, but no blocker. Would be good if @mui/material-ui could take a look as well, for awareness.

@Janpot Janpot requested a review from a team February 13, 2026 11:17
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Feb 13, 2026
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Feb 19, 2026
@brijeshb42
Copy link
Contributor Author

@siriwatknp Could you take a look at the PR.

Copy link
Member

@siriwatknp siriwatknp left a comment

Choose a reason for hiding this comment

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

I trust the eslint rule 🙏

@brijeshb42 brijeshb42 merged commit 09f4536 into mui:master Feb 19, 2026
23 checks passed
@brijeshb42 brijeshb42 deleted the opt-undefined branch February 19, 2026 09:14
@brijeshb42 brijeshb42 added v7.x needs cherry-pick The PR should be cherry-picked to master after merge. labels Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs cherry-pick The PR should be cherry-picked to master after merge. scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). v7.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TypeScript option --exactOptionalPropertyTypes is not supported

4 participants