Skip to content

ref(flags): Remove organizations:uptime-auto-detected-monitor-emails#114806

Open
wedamija wants to merge 1 commit intomasterfrom
flag-cleanup/organizations-uptime-auto-detected-monitor-emails
Open

ref(flags): Remove organizations:uptime-auto-detected-monitor-emails#114806
wedamija wants to merge 1 commit intomasterfrom
flag-cleanup/organizations-uptime-auto-detected-monitor-emails

Conversation

@wedamija
Copy link
Copy Markdown
Member

@wedamija wedamija commented May 4, 2026

The flag scanner classified this as flagpole-disabled. The flag gated the entire send_auto_detected_notifications task, so the email path is unreachable. Removes the task, template, debug view, result_handler dispatch, and test coverage wholesale.

@wedamija wedamija requested review from a team as code owners May 4, 2026 23:09
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 4, 2026
The flag scanner classified this as flagpole-disabled. The flag gated
the entire send_auto_detected_notifications task, so the email path is
unreachable. Removes the task, template, debug view, result_handler
dispatch, and test coverage wholesale.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@wedamija wedamija force-pushed the flag-cleanup/organizations-uptime-auto-detected-monitor-emails branch from cb49408 to 4918a36 Compare May 4, 2026 23:11
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Backend Test Failures

Failures on eb41a21 in this run:

tests/sentry/test_mypy_stronglist.py::test_stronglistlog
[gw0] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/test_mypy_stronglist.py:20: in test_stronglist
    raise AssertionError(f"\n\n{proc.stdout}")
E   AssertionError: 
E   
E   pyproject.toml: sentry.web.frontend.debug.debug_uptime_auto_detected_monitor_email in weaklist does not match any files!

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.

2 participants