Skip to content

Conversation

bsekachev
Copy link
Member

@bsekachev bsekachev commented Jul 25, 2025

Motivation and context

To prevent incorrect configuration passed to the server.
It would be nice to implement similar checks on server level, however this patch is only to enhance UI.

How has this been tested?

Checklist

  • I submit my changes into the develop branch
  • I have created a changelog fragment
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@bsekachev bsekachev requested a review from nmanovic as a code owner July 25, 2025 10:55
Copy link

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 50.98039% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.72%. Comparing base (f2241f9) to head (e38351c).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9677      +/-   ##
===========================================
- Coverage    74.36%   73.72%   -0.64%     
===========================================
  Files          455      400      -55     
  Lines        47359    44422    -2937     
  Branches      3963     3974      +11     
===========================================
- Hits         35217    32751    -2466     
+ Misses       12142    11671     -471     
Components Coverage Δ
cvat-ui 77.62% <50.98%> (-0.08%) ⬇️
cvat-server 70.37% <ø> (-1.44%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bsekachev bsekachev merged commit 7d2ac8e into develop Jul 29, 2025
38 checks passed
@bsekachev bsekachev deleted the bs/enhanced_labels_validation branch July 29, 2025 17:54
@cvat-bot cvat-bot bot mentioned this pull request Aug 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants