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(bitbucket-server): CODEOWNERS support and stacktrace linking #86639

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jianyuan
Copy link
Contributor

@jianyuan jianyuan commented Mar 7, 2025

Bitbucket Server CODEOWNERS and stacktrace linking support. Note that the API URLs for Bitbucket Server are slightly different that that for Bitbucket Cloud.

Screenshots Screenshot 2025-03-11 at 23 24 47 Screenshot 2025-03-11 at 23 24 24 Screenshot 2025-03-11 at 23 24 36 Screenshot 2025-03-11 at 23 24 58 Screenshot 2025-03-11 at 23 24 55

@jianyuan jianyuan requested review from a team as code owners March 7, 2025 23:07
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 7, 2025
@jianyuan jianyuan marked this pull request as draft March 10, 2025 08:02
@cathteng cathteng added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 10, 2025
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 0b412c3 to 6a322ee Compare March 10, 2025 21:47
@github-actions github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 10, 2025
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch 2 times, most recently from f0f7151 to 6d4f32c Compare March 10, 2025 22:01
@jianyuan jianyuan marked this pull request as ready for review March 10, 2025 22:21
parsed_url = urlparse(url)
qs = parse_qs(parsed_url.query)

if "at" in qs and len(qs["at"]) == 1:
Copy link
Member

Choose a reason for hiding this comment

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

could you explain what the "at" means here? i am curious how this is different from bitbucket

Copy link
Member

Choose a reason for hiding this comment

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

i see, at refers to the branch i think

Comment on lines +358 to +362
if repo.url is None:
return ""
parsed_repo_url = urlparse(repo.url)
parsed_url = urlparse(url)
return parsed_url.path.replace(parsed_repo_url.path + "/", "")
Copy link
Member

Choose a reason for hiding this comment

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

i'm also curious how this is different from bitbucket, it kind of looks the same?

@cathteng cathteng added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 11, 2025
@cathteng
Copy link
Member

can you add any screenshots / videos to show this works?

@github-actions github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 11, 2025
@jianyuan
Copy link
Contributor Author

@cathteng done! check out the PR description.

@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 3a06de3 to 357114f Compare March 18, 2025 18:27
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