Skip to content

fix(weekly-report): filter to active teams for user projects #91997

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JoshFerge
Copy link
Member

I'm investigating #73594 and as part of that, we should filter out pending deletion team relations from the projects included in the email.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 20, 2025
@JoshFerge JoshFerge requested review from a team May 20, 2025 23:22
@JoshFerge JoshFerge enabled auto-merge (squash) May 20, 2025 23:22
Copy link

codecov bot commented May 21, 2025

Codecov Report

Attention: Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.

⚠️ Parser warning

The parser emitted a warning. Please review your JUnit XML file:

Warning while parsing testcase attributes: Limit of string is 1000 chars, for name, we got 2083 at 1:157341 in /home/runner/work/sentry/sentry/.artifacts/pytest.junit.xml
Files with missing lines Patch % Lines
tests/sentry/tasks/test_weekly_reports.py 76.92% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #91997      +/-   ##
==========================================
- Coverage   87.62%   86.49%   -1.13%     
==========================================
  Files       10355    10356       +1     
  Lines      587429   587629     +200     
  Branches    22586    22586              
==========================================
- Hits       514723   508258    -6465     
- Misses      72279    78944    +6665     
  Partials      427      427              

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants