Skip to content
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

feat(azure-devops): CODEOWNERS support #86958

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jianyuan
Copy link
Contributor

CODEOWNERS support for Azure DevOps

Screenshots Screenshot 2025-03-12 at 00 03 43 Screenshot 2025-03-12 at 00 03 56 Screenshot 2025-03-12 at 23 05 39

@jianyuan jianyuan requested review from a team as code owners March 12, 2025 23:12
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 12, 2025
@iamrajjoshi iamrajjoshi added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 12, 2025
Copy link
Member

@iamrajjoshi iamrajjoshi left a comment

Choose a reason for hiding this comment

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

lgtm, just a question

api_version=self.api_version,
method=method,
api_version_preview=self.api_version_preview if api_preview else "",
)
kwargs.setdefault("headers", {}).update(new_headers)
kwargs["headers"] = {**base_headers, **(kwargs.get("headers", {}))}
Copy link
Member

Choose a reason for hiding this comment

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

curious why this change was required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

prepare_headers() returns a dictionary with an Accept key. I need to override the header to Accept: */* in the following method. I thought allowing user-provided headers to take precedence would make sense.

https://github.com/getsentry/sentry/pull/86958/files#diff-2fe5f1304407937661dd4a7be31cc76c586a7dfd672a87dce18195424826c4e6R464

@iamrajjoshi iamrajjoshi requested a review from a team March 12, 2025 23:33
@jianyuan jianyuan force-pushed the feat/codeowners-azure-devops branch from 03e7ca2 to 64d53a3 Compare March 18, 2025 18:28
@github-actions github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 18, 2025
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