Skip to content

[DOP-24446] change uuid6 to default uuid in type annotation#194

Merged
TiGrib merged 2 commits intodevelopfrom
feature/DOP-24446
Mar 24, 2025
Merged

[DOP-24446] change uuid6 to default uuid in type annotation#194
TiGrib merged 2 commits intodevelopfrom
feature/DOP-24446

Conversation

@TiGrib
Copy link
Contributor

@TiGrib TiGrib commented Mar 24, 2025

Change Summary

Remove uuid6 from annotations inside server. Also remove type-ignore from 'routes'.

Related issue number

[DOP-24446]

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@TiGrib TiGrib requested a review from dolfinus March 24, 2025 09:06
@TiGrib TiGrib self-assigned this Mar 24, 2025
@codecov
Copy link

codecov bot commented Mar 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.88%. Comparing base (4cdc96e) to head (06a1988).
Report is 112 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #194      +/-   ##
===========================================
- Coverage    91.88%   91.88%   -0.01%     
===========================================
  Files          199      199              
  Lines         4954     4953       -1     
  Branches       343      343              
===========================================
- Hits          4552     4551       -1     
  Misses         334      334              
  Partials        68       68              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dolfinus dolfinus added the ci:skip-changelog Add this label to skip changelog file check label Mar 24, 2025
@TiGrib TiGrib force-pushed the feature/DOP-24446 branch from 4f0a9ef to 06a1988 Compare March 24, 2025 12:15
@TiGrib TiGrib merged commit d9cce6b into develop Mar 24, 2025
13 checks passed
@TiGrib TiGrib deleted the feature/DOP-24446 branch March 24, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip-changelog Add this label to skip changelog file check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants