-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Extend the shared library to provide more data for Bitbucket PRs #4128
Comments
This was referenced Mar 5, 2025
Merged
Here is a ticket related to making some changes in the shared library. What do you think is the best way to handle it? |
sergeibbb
added a commit
that referenced
this issue
Mar 10, 2025
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128
sergeibbb
added a commit
that referenced
this issue
Mar 10, 2025
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128
7 tasks
sergeibbb
added a commit
that referenced
this issue
Mar 10, 2025
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
sergeibbb
added a commit
that referenced
this issue
Mar 10, 2025
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
sergeibbb
added a commit
that referenced
this issue
Mar 11, 2025
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
sergeibbb
added a commit
that referenced
this issue
Mar 11, 2025
Integrates changes from gitkraken/provider-apis-package-js#223 Solves #4128 (#4141)
7 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
There is a getPullRequestsForUserAndWorkspace method in the shared library. But in #4099 we used the own implementation of it, because the shared library does not provide info about reviewers.
Also we need URLs of head and base repositories, to avoid hacking it like here:
Our current Bitbucket conversion of type is here, all information is collected in one request:
vscode-gitlens/src/plus/integrations/providers/bitbucket/models.ts
Lines 366 to 397 in 289fb9e
Goals:
Implementation
The text was updated successfully, but these errors were encountered: