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

[test] Fix AdapterDayjs coverage calculation #14957

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

LukasTy
Copy link
Member

@LukasTy LukasTy commented Oct 14, 2024

Note

This is to unblock the existing failing PRs.

Ignore this line in test coverage calculation which started happening after #13919 got merged.

We should be able to remove the ignore once we bump the dayjs peer dependency version on the next major and simplify the code.

@LukasTy LukasTy added test core Infrastructure work going on behind the scenes component: pickers This is the name of the generic UI component, not the React module! labels Oct 14, 2024
@LukasTy LukasTy requested review from a team October 14, 2024 10:40
@LukasTy LukasTy self-assigned this Oct 14, 2024
@mui-bot
Copy link

mui-bot commented Oct 14, 2024

Deploy preview: https://deploy-preview-14957--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 16e9e03

@LukasTy LukasTy merged commit 46df966 into mui:master Oct 14, 2024
24 checks passed
@LukasTy LukasTy deleted the fix-adapter-day-js-coverage branch October 14, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: pickers This is the name of the generic UI component, not the React module! core Infrastructure work going on behind the scenes test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants