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

Bump @types/react to v18.2.45 #40146

Merged
merged 3 commits into from
Dec 28, 2023
Merged

Bump @types/react to v18.2.45 #40146

merged 3 commits into from
Dec 28, 2023

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Dec 10, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
@types/react (source) ^18.2.42 -> ^18.2.45 age adoption passing confidence

Configuration

📅 Schedule: Branch creation - "on sunday before 6:00am" in timezone UTC, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the dependencies Update of dependencies label Dec 10, 2023
@mui-bot
Copy link

mui-bot commented Dec 10, 2023

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 85eb97d

@LukasTy
Copy link
Member

LukasTy commented Dec 12, 2023

The issue regarding the removal of placeholder field is discussed at mui/mui-x#11280.

NumberInputOwnerState
>;
export type NumberInputOwnProps = Omit<UseNumberInputParameters, 'error'> &
Pick<React.InputHTMLAttributes<HTMLInputElement>, 'placeholder'> & {
Copy link
Member

Choose a reason for hiding this comment

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

I wanted to add it to AdditionalProps in NumberInputTypeMap, but then it's not picked up for prop types generation. 🙈
Do you prefer manually documenting this prop instead, similar to error? 🤔

@LukasTy LukasTy requested review from mj12albert and a team December 12, 2023 08:51
@LukasTy
Copy link
Member

LukasTy commented Dec 12, 2023

@mui/base-ui @mj12albert could you please review the proposed change and adjust if necessary? 🤔
This problem could be an issue for someone using @mui/material and not excluding it from TS, as noted here: mui/mui-x#11280 (comment)

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Dec 25, 2023
@mj12albert mj12albert self-assigned this Dec 28, 2023
@renovate renovate bot changed the title Bump @types/react to v18.2.43 Bump @types/react to v18.2.45 Dec 28, 2023
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Dec 28, 2023
Copy link
Contributor Author

renovate bot commented Dec 28, 2023

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@mj12albert
Copy link
Member

@LukasTy Sorry I ticked the rebase checkbox and wiped out your changes 🙈

But I've inlined it back into OwnProps to keep it flatter looking 😬

Copy link
Member

@LukasTy LukasTy left a comment

Choose a reason for hiding this comment

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

LukasTy Sorry I ticked the rebase checkbox and wiped out your changes 🙈

But I've inlined it back into OwnProps to keep it flatter looking 😬

No problem. 👌
LGTM.
Sure, whatever you think works best.
IMHO, placeholder prop is self explanatory and having it without JSDoc wouldn't hurt. However, extra clarity won't hurt. 👍

@mj12albert mj12albert merged commit e13f5d0 into master Dec 28, 2023
22 checks passed
@mj12albert mj12albert deleted the renovate/react-18.x branch December 28, 2023 16:56
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Jan 9, 2024
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Albert Yu <albert@albertyu.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Update of dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants