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
Feat/toggleSelectOptions #252
Feat/toggleSelectOptions #252
Conversation
Codecov Report
@@ Coverage Diff @@
## master #252 +/- ##
==========================================
+ Coverage 99.20% 99.26% +0.05%
==========================================
Files 2 2
Lines 253 273 +20
Branches 65 70 +5
==========================================
+ Hits 251 271 +20
Misses 2 2
Continue to review full report at Codecov.
|
Hi @malcolm-kee, So sorry that you put all of this work into this PR and haven't heard anything in months. I'm starting to pick up development here. Would you be interested in getting this PR updated with the latest version of the code? If so, there are two bits of feedback:
Here's an example usage: user-event/src/__tests__/type-modifiers.js Lines 19 to 29 in 4abecfa
Again, I apologize for the trouble. |
@kentcdodds no problem at all. I'll add the 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.
This is solid! Just a few suggestions/ideas.
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.
Super! Thank you!
@all-contributors please add @malcolm-kee for code, docs, and tests |
I've put up a pull request to add @malcolm-kee! 🎉 |
Thanks so much for your help! I've added you as a collaborator on the project. Please make sure that you review the |
Thanks @kentcdodds ! |
🎉 This PR is included in version 11.2.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
resolves #180