Skip to content

fix(weekly_report): Switch to uuidv4 #55366

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

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Conversation

scttcper
Copy link
Member

switch to uuidv4 like the other notifications

switch to uuidv4 like the other alerts
@scttcper scttcper requested a review from jangjodi August 29, 2023 17:20
@scttcper scttcper requested a review from a team as a code owner August 29, 2023 17:20
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 29, 2023
@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Merging #55366 (bf3d58b) into master (bd19446) will increase coverage by 0.00%.
Report is 8 commits behind head on master.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #55366   +/-   ##
=======================================
  Coverage   80.01%   80.01%           
=======================================
  Files        5041     5041           
  Lines      214935   214935           
  Branches    36494    36495    +1     
=======================================
+ Hits       171973   171976    +3     
+ Misses      37680    37678    -2     
+ Partials     5282     5281    -1     
Files Changed Coverage Δ
src/sentry/notifications/utils/__init__.py 73.03% <ø> (-0.33%) ⬇️
src/sentry/tasks/weekly_reports.py 77.54% <100.00%> (ø)

... and 5 files with indirect coverage changes

@scttcper scttcper merged commit 8d630f8 into master Aug 29, 2023
@scttcper scttcper deleted the scttcper/weekly-report-uuid branch August 29, 2023 18:09
@github-actions github-actions bot locked and limited conversation to collaborators Sep 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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