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 TS 4.7 compat issues #32041

Merged
merged 1 commit into from Mar 30, 2022
Merged

[core] Fix TS 4.7 compat issues #32041

merged 1 commit into from Mar 30, 2022

Conversation

eps1lon
Copy link
Member

@eps1lon eps1lon commented Mar 29, 2022

Targetting 4.x but similar changes are probably required for current typings as well.

Fixes issues with TypeScript 4.7 canary (microsoft/TypeScript#48366 specifically) that are blocking DefinitelyTyped/DefinitelyTyped#56210

@eps1lon eps1lon marked this pull request as ready for review March 29, 2022 19:55
@eps1lon eps1lon requested a review from a team March 29, 2022 19:55
@siriwatknp
Copy link
Member

What's wrong with argos 🤔

@siriwatknp
Copy link
Member

Targetting 4.x but similar changes are probably required for current typings as well.

Fixes issues with TypeScript 4.7 canary (microsoft/TypeScript#48366 specifically) that are blocking DefinitelyTyped/DefinitelyTyped#56210

What's about other components eg. Toolbar? Do they need changes?

@eps1lon
Copy link
Member Author

eps1lon commented Mar 30, 2022

What's wrong with argos 🤔

Argos compares against master. Failure on older branches is expected.

@eps1lon
Copy link
Member Author

eps1lon commented Mar 30, 2022

Targetting 4.x but similar changes are probably required for current typings as well.

Fixes issues with TypeScript 4.7 canary (microsoft/TypeScript#48366 specifically) that are blocking DefinitelyTyped/DefinitelyTyped#56210

What's about other components eg. Toolbar? Do they need changes?

Other components did not error.

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.

👍

@eps1lon eps1lon merged commit ac08a3a into mui:v4.x Mar 30, 2022
@eps1lon eps1lon deleted the fix/ts-4.7-compat branch March 30, 2022 06:54
This was referenced Mar 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants