Skip to content

ref(repos): Remove old SCM integration tree view#115175

Merged
evanpurkhiser merged 1 commit intomasterfrom
evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view
May 8, 2026
Merged

ref(repos): Remove old SCM integration tree view#115175
evanpurkhiser merged 1 commit intomasterfrom
evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view

Conversation

@evanpurkhiser
Copy link
Copy Markdown
Member

Deletes the ScmIntegrationTree component and all sub-files now that the v2 repositories page is the only experience. Reduces the repositories settings index to a re-export of OrganizationRepositoriesV2.

@evanpurkhiser evanpurkhiser requested review from a team as code owners May 8, 2026 15:12
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.46%

@evanpurkhiser evanpurkhiser marked this pull request as draft May 8, 2026 15:16
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from a048927 to f78777f Compare May 8, 2026 17:33
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from f78777f to f10b19e Compare May 8, 2026 17:39
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

Backend Test Failures

Failures on 126041e in this run:

tests/sentry/integrations/perforce/test_pipeline.py::PerforceApiPipelineTest::test_successful_connectionlog
[gw1] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/perforce/test_pipeline.py:68: in test_successful_connection
    assert resp.status_code == 200
E   assert 400 == 200
E    +  where 400 = <Response status_code=400, "application/json">.status_code
tests/sentry/integrations/perforce/test_pipeline.py::PerforceApiPipelineTest::test_auth_failure_returns_errorlog
[gw1] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/perforce/test_pipeline.py:92: in test_auth_failure_returns_error
    assert "Authentication failed" in resp.data["data"]["detail"]
E   KeyError: 'data'
tests/sentry/integrations/perforce/test_pipeline.py::PerforceApiPipelineTest::test_connection_failure_returns_errorlog
[gw1] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/perforce/test_pipeline.py:115: in test_connection_failure_returns_error
    assert "Failed to connect" in resp.data["data"]["detail"]
E   KeyError: 'data'

@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from f10b19e to 035aa30 Compare May 8, 2026 18:03
@evanpurkhiser evanpurkhiser marked this pull request as ready for review May 8, 2026 18:03
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from 035aa30 to e1b1a2e Compare May 8, 2026 18:41
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from ed01c8e to 40ee21f Compare May 8, 2026 18:48
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from 40ee21f to 6dd2d19 Compare May 8, 2026 18:52
@evanpurkhiser evanpurkhiser enabled auto-merge (squash) May 8, 2026 18:53
Deletes the ScmIntegrationTree component and all sub-files now that
the v2 repositories page is the only experience. Reduces the
repositories settings index to a re-export of OrganizationRepositoriesV2.
@evanpurkhiser evanpurkhiser force-pushed the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch from 6dd2d19 to 2d0cd4b Compare May 8, 2026 18:59
@evanpurkhiser evanpurkhiser merged commit 590d479 into master May 8, 2026
71 checks passed
@evanpurkhiser evanpurkhiser deleted the evanpurkhiser/ref-repos-remove-old-scm-integration-tree-view branch May 8, 2026 19:06
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 Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants