-
Notifications
You must be signed in to change notification settings - Fork 13
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: add tests and fix accessibility issues for the select #462
Conversation
1d55038
to
6af8f4e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #462 +/- ##
==========================================
- Coverage 91.64% 89.90% -1.74%
==========================================
Files 101 108 +7
Lines 2249 3954 +1705
Branches 406 797 +391
==========================================
+ Hits 2061 3555 +1494
- Misses 119 397 +278
+ Partials 69 2 -67
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
6af8f4e
to
d54d780
Compare
3e810f3
to
ceacbdb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fbasso left some minor comments, thx for the PR !
ceacbdb
to
8a58ec5
Compare
@quentinderoubaix, pushed again with changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the changes @fbasso !
8a58ec5
to
0991f2a
Compare
Thanks @quentinderoubaix |
fix #459
fix #263