Skip to content

ref(scm): Wrap raw response with data key and provide the headers#111715

Merged
cmanallen merged 4 commits intomasterfrom
cmanallen/scm-wrap-raw-response
Mar 27, 2026
Merged

ref(scm): Wrap raw response with data key and provide the headers#111715
cmanallen merged 4 commits intomasterfrom
cmanallen/scm-wrap-raw-response

Conversation

@cmanallen
Copy link
Copy Markdown
Member

This is a forward compatibility step. By wrapping the raw response with the data key we can add more keys later to communicate more information (if needed) without breaking existing callers. I decided to add the headers to the raw response for completeness.

@cmanallen cmanallen requested a review from a team as a code owner March 27, 2026 15:45
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 27, 2026
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 src/sentry/scm/private/providers/github.py Outdated
@cmanallen cmanallen merged commit 5a8400b into master Mar 27, 2026
64 of 65 checks passed
@cmanallen cmanallen deleted the cmanallen/scm-wrap-raw-response branch March 27, 2026 16:41
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants