-
-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
feat(bitbucket-server): CODEOWNERS support and stacktrace linking #86639
Conversation
0b412c3
to
6a322ee
Compare
f0f7151
to
6d4f32c
Compare
parsed_url = urlparse(url) | ||
qs = parse_qs(parsed_url.query) | ||
|
||
if "at" in qs and len(qs["at"]) == 1: |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
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 + "/", "") |
There was a problem hiding this comment.
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?
can you add any screenshots / videos to show this works? |
@cathteng done! check out the PR description. |
3a06de3
to
357114f
Compare
357114f
to
4c383ac
Compare
4c383ac
to
4df64f5
Compare
4df64f5
to
3f9dc48
Compare
Co-authored-by: Cathy Teng <70817427+cathteng@users.noreply.github.com>
Co-authored-by: Cathy Teng <70817427+cathteng@users.noreply.github.com>
3f9dc48
to
65d0804
Compare
65d0804
to
c5069d2
Compare
revert failed (conflict? already reverted?) -- check the logs |
Head branch was pushed to by a user without write access
Bitbucket Server CODEOWNERS and stacktrace linking support. Note that the API URLs for Bitbucket Server are slightly different that that for Bitbucket Cloud.
Screenshots