Skip to content

Feat: basic support for bitucket #650

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
merged 12 commits into from
Apr 12, 2025
Merged

Conversation

ZibanPirate
Copy link
Member

@ZibanPirate ZibanPirate commented Apr 12, 2025

We were asked in #648. Since we added the Open-listings project #649 from Bitbucket, it makes sense to now support at least some basic endpoints from Bitbucket.

  • added Bitbucket service with two methods for now
  • Bitbucket.getRepository to get general repo info
  • and Bitbucket.listRepositoryContributors
  • updated cron job to account for multi-providers (Github and bitbucket for now)

you can test this on stage:

  • Bug fix
  • New feature
  • Other

@github-actions github-actions bot added web Web related changes ( web folder ) api API related changes ( api folder ) models models package labels Apr 12, 2025
@github-actions github-actions bot added the feature Feature label Apr 12, 2025
@ZibanPirate ZibanPirate marked this pull request as ready for review April 12, 2025 09:45
@ZibanPirate ZibanPirate requested a review from a team April 12, 2025 09:45
@github-actions github-actions bot added the repo label Apr 12, 2025
omdxp
omdxp previously approved these changes Apr 12, 2025
Copy link
Member

@omdxp omdxp left a comment

Choose a reason for hiding this comment

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

Maybe we should start applying Hexagonal architecture (Ports and Adapters) from now on

@github-project-automation github-project-automation bot moved this to Approved in DzCode i/o Apr 12, 2025
@ZibanPirate ZibanPirate merged commit d3cab58 into main Apr 12, 2025
24 checks passed
@ZibanPirate ZibanPirate deleted the feat/basic-support-for-bitucket branch April 12, 2025 10:41
@github-project-automation github-project-automation bot moved this from Approved to Merged in DzCode i/o Apr 12, 2025
username?: string;
display_name: string;
nickname?: string;
links: { avatar: { href: "https://bitbucket.org/account/open-listings/avatar/" } };
Copy link
Contributor

Choose a reason for hiding this comment

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

@ZibanPirate careful about link to avatar. It does not look like static.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api API related changes ( api folder ) feature Feature models models package repo web Web related changes ( web folder )
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants