feat: convert multiple_selected_dates.test.js into tsx #4827
Conversation
|
Nice one, i've started the CI and your PR will be reviewed by the @PullRequestInc team. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4827 +/- ##
=======================================
Coverage 96.67% 96.67%
=======================================
Files 28 28
Lines 2977 2977
Branches 1285 1285
=======================================
Hits 2878 2878
Misses 99 99 ☔ View full report in Codecov by Sentry. |
![pullrequest[bot]](https://avatars.githubusercontent.com/in/3434?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ This pull request was sent to the PullRequest network for review. Expert reviewers are now being matched to your request based on the code's requirements. Stay tuned!
What to expect from this code review:
- Comments posted to any areas of potential concern or improvement.
- Detailed feedback or actions needed to resolve issues that are found.
- Turnaround times vary, but we aim to be swift.
@yasuaki640 you can click here to see the review status or cancel the code review job - or - cancel by adding [!pr] to the title of the pull request.
There was a problem hiding this comment.
The changes look good to me and help improve the type safety of the multiple selected dates tests. No blocking issues or security concerns here. Thanks!
Reviewed with ❤️ by PullRequest
name: Migrate filter_times_test.test.js
Description
Linked issue: #4700
Changes
Converted multiple_selected_dates.test.js to a tsx file.
Contribution checklist