Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore(superuser): Add logger to to check org id #61814

Merged
merged 2 commits into from
Dec 15, 2023
Merged

chore(superuser): Add logger to to check org id #61814

merged 2 commits into from
Dec 15, 2023

Conversation

schew2381
Copy link
Contributor

No description provided.

@schew2381 schew2381 requested review from cathteng and a team December 14, 2023 23:13
@schew2381 schew2381 self-assigned this Dec 14, 2023
@schew2381 schew2381 requested a review from a team as a code owner December 14, 2023 23:13
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 14, 2023
@schew2381 schew2381 enabled auto-merge (squash) December 14, 2023 23:13
@codecov Codecov
Copy link

codecov bot commented Dec 15, 2023 •
edited
Loading

Codecov Report

Merging #61814 (cb8e85a) into master (17b7a79) will decrease coverage by 0.01%.
Report is 2 commits behind head on master.
The diff coverage is 100.00%.

❗ Current head cb8e85a differs from pull request most recent head 4492782. Consider uploading reports for the commit 4492782 to get more accurate results

Additional details and impacted files 
@@            Coverage Diff             @@
##           master   #61814      +/-   ##
==========================================
- Coverage   81.19%   81.19%   -0.01%     
==========================================
  Files        5187     5187              
  Lines      228387   228381       -6     
  Branches    38311    38310       -1     
==========================================
- Hits       185445   185435      -10     
- Misses      37301    37302       +1     
- Partials     5641     5644       +3
Files Coverage Δ
src/sentry/auth/superuser.py 88.58% <100.00%> (+0.05%) ⬆️

... and 23 files with indirect coverage changes

@schew2381 schew2381 force-pushed the seiji/chore/add-logger-to-check-org-id branch from 3c3465d to 4492782 Compare December 15, 2023 18:52
@schew2381 schew2381 merged commit d16b5d4 into master Dec 15, 2023
48 checks passed
@schew2381 schew2381 deleted the seiji/chore/add-logger-to-check-org-id branch December 15, 2023 19:13
@schew2381 schew2381 added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Dec 15, 2023
@getsentry-bot
Copy link
Contributor

PR reverted: 6bf39b4

getsentry-bot added a commit that referenced this pull request Dec 15, 2023
@getsentry-bot @schew2381
Revert "chore(superuser): Add logger to to check org id (#61814)"
6bf39b4 
This reverts commit d16b5d4.

Co-authored-by: schew2381 <67301797+schew2381@users.noreply.github.com>
@github-actions github-actions bot locked and limited conversation to collaborators Dec 31, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cathteng cathteng cathteng approved these changes

Assignees

@schew2381 schew2381

Labels
Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@schew2381 @getsentry-bot @cathteng