-
Notifications
You must be signed in to change notification settings - Fork 3.3k
fix: BROS-352: Revert order of returned params in useCopyText()
#8238
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
Conversation
This order was not supported by all usages.
✅ Deploy Preview for label-studio-playground canceled.
|
✅ Deploy Preview for label-studio-storybook canceled.
|
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
✅ Deploy Preview for heartex-docs canceled.
|
bmartel
left a comment
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.
Are we sure these are the only affected usages?
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8238 +/- ##
===========================================
- Coverage 70.58% 66.52% -4.07%
===========================================
Files 725 509 -216
Lines 52670 34612 -18058
Branches 9015 9015
===========================================
- Hits 37179 23024 -14155
+ Misses 15488 11585 -3903
Partials 3 3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I checked the other usages, this will work. We need to address the inconsistent usage of this, tuples should always be [state, setState] when creating hooks like this. |
|
/git merge
|
This order was not supported by all usages.