Skip to content

feat(bitbucket-server): CODEOWNERS support and stacktrace linking #86639

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

Merged

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 +357
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
@iamrajjoshi iamrajjoshi added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 23, 2025
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 357114f to 4c383ac Compare March 23, 2025 12:51
@github-actions github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Mar 23, 2025
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 4c383ac to 4df64f5 Compare March 23, 2025 12:52
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 4df64f5 to 3f9dc48 Compare March 31, 2025 22:10
jianyuan and others added 3 commits April 21, 2025 19:26
Co-authored-by: Cathy Teng <70817427+cathteng@users.noreply.github.com>
Co-authored-by: Cathy Teng <70817427+cathteng@users.noreply.github.com>
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 3f9dc48 to 65d0804 Compare April 21, 2025 18:26
@jianyuan jianyuan force-pushed the feat/codeowners-bitbucket-server branch from 65d0804 to c5069d2 Compare April 21, 2025 18:39
@iamrajjoshi iamrajjoshi added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Apr 21, 2025
@iamrajjoshi iamrajjoshi enabled auto-merge (squash) April 21, 2025 18:41
@getsentry-bot
Copy link
Contributor

revert failed (conflict? already reverted?) -- check the logs

@iamrajjoshi iamrajjoshi added Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests and removed Trigger: Revert Add to a merged PR to revert it (skips CI) labels Apr 21, 2025
auto-merge was automatically disabled April 30, 2025 09:52

Head branch was pushed to by a user without write access

@github-actions github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Apr 30, 2025
@iamrajjoshi iamrajjoshi added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 1, 2025
@github-actions github-actions bot removed the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 1, 2025
@iamrajjoshi iamrajjoshi added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label May 1, 2025
@cathteng cathteng merged commit 5362d14 into getsentry:master May 1, 2025
59 of 60 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2025
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 Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants