Skip to content

fix(migrations): Actually fix BoundedPositiveBigIntegerField #83431

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

wedamija
Copy link
Member

Third time's the charm!

@wedamija wedamija requested review from brendanhsentry and a team January 14, 2025 20:21
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 14, 2025
@wedamija wedamija enabled auto-merge (squash) January 14, 2025 20:22
@wedamija wedamija merged commit 90afd07 into master Jan 14, 2025
49 checks passed
@wedamija wedamija deleted the danf/migrations-third-time branch January 14, 2025 20:50
Copy link

codecov bot commented Jan 14, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/db/models/fields/bounded.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #83431      +/-   ##
==========================================
+ Coverage   87.51%   87.52%   +0.01%     
==========================================
  Files        9405     9405              
  Lines      537279   537279              
  Branches    21140    21140              
==========================================
+ Hits       470177   470235      +58     
+ Misses      66751    66693      -58     
  Partials      351      351              

andrewshie-sentry pushed a commit that referenced this pull request Jan 22, 2025
Third time's the charm!

<!-- Describe your PR here. -->
@github-actions github-actions bot locked and limited conversation to collaborators Jan 30, 2025
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants