Skip to content

Conversation

@hlomzik
Copy link
Collaborator

@hlomzik hlomzik commented Aug 22, 2025

This order was not supported by all usages.

image (11)

This order was not supported by all usages.
@netlify
Copy link

netlify bot commented Aug 22, 2025

Deploy Preview for label-studio-playground canceled.

Name Link
🔨 Latest commit f5c2fec
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/68a8e0c0d31f850008512108

@netlify
Copy link

netlify bot commented Aug 22, 2025

Deploy Preview for label-studio-storybook canceled.

Name Link
🔨 Latest commit f5c2fec
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/68a8e0c053c6090009f1b503

@hlomzik hlomzik requested a review from nick-skriabin August 22, 2025 19:02
@netlify
Copy link

netlify bot commented Aug 22, 2025

Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit f5c2fec
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/68a8e0c04a684d0008114d50

@hlomzik hlomzik requested a review from Gondragos August 22, 2025 19:02
@github-actions github-actions bot added the fix label Aug 22, 2025
@netlify
Copy link

netlify bot commented Aug 22, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit f5c2fec
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/68a8e0c2f065b300083c2f8b

Copy link
Contributor

@bmartel bmartel left a 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
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 66.52%. Comparing base (def47a2) to head (f5c2fec).
⚠️ Report is 29 commits behind head on develop.

Files with missing lines Patch % Lines
web/libs/ui/src/lib/code-block/code-block.tsx 0.00% 1 Missing ⚠️
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              
Flag Coverage Δ
lsf-e2e 60.02% <50.00%> (+0.03%) ⬆️
lsf-integration 56.27% <50.00%> (-0.05%) ⬇️
lsf-unit 9.40% <ø> (ø)
pytests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bmartel
Copy link
Contributor

bmartel commented Aug 22, 2025

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.

@bmartel
Copy link
Contributor

bmartel commented Aug 22, 2025

/git merge

Workflow run
Successfully merged: 3 files changed, 15 insertions(+), 9 deletions(-)

@robot-ci-heartex robot-ci-heartex merged commit c828b85 into develop Aug 28, 2025
44 of 46 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-bros-352/copied branch August 28, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants