Skip to content

ci(st): handle renames#111937

Merged
joshuarli merged 3 commits intomasterfrom
ci-st-handle-renames
Mar 31, 2026
Merged

ci(st): handle renames#111937
joshuarli merged 3 commits intomasterfrom
ci-st-handle-renames

Conversation

@joshuarli
Copy link
Copy Markdown
Member

see https://github.com/getsentry/getsentry/pull/19716

these changes don't affect existing infra

…t coverage DB

When a file is renamed, the coverage DB still stores the old path. Without
this fix, the new filename gets zero hits and no relevant tests run.

Adds --previous-filenames to compute-sentry-selected-tests.py, wires
sentryPreviousFilenames through getsentry-dispatch.js, and collects
previous_filename from the GitHub PR files API in the workflows that
call these scripts (done in the next commit).

Co-authored-by: Claude <noreply@anthropic.com>
@joshuarli joshuarli requested a review from a team as a code owner March 31, 2026 19:14
@joshuarli joshuarli requested a review from rbro112 March 31, 2026 19:14
@github-actions github-actions bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Mar 31, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🚨 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.

Comment thread .github/workflows/scripts/compute-sentry-selected-tests.py
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment thread .github/workflows/scripts/getsentry-dispatch.js
@joshuarli joshuarli merged commit 92c4bbd into master Mar 31, 2026
86 checks passed
@joshuarli joshuarli deleted the ci-st-handle-renames branch March 31, 2026 23:00
dashed pushed a commit that referenced this pull request Apr 1, 2026
see getsentry/getsentry#19716

these changes don't affect existing infra
@github-actions github-actions bot locked and limited conversation to collaborators Apr 16, 2026
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 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