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: add github client dedicated for ci #186

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vohoanglong0107
Copy link
Contributor

What

This PR adds a new github client to interact with CI repo.

Why

The CI repo for Rust belongs to a different organization than the original repo, since we can't create a fork of a repo inside the same organization. However, Github doesn't support cross-orgs operations with Github App. We have to use another mechanism to authenticate with the CI repo.

How

This PR introduces a new Github client, authenticated with Personal Access Token. This is the same method used by homu (ref)

@vohoanglong0107 vohoanglong0107 marked this pull request as ready for review December 8, 2024 01:45
@Kobzol
Copy link
Contributor

Kobzol commented Dec 9, 2024

Oh, crap, that is unfortunate. I'll discuss this with the infra team.

@Skgland
Copy link

Skgland commented Dec 9, 2024

The CI repo for Rust belongs to a different organization than the original repo, since we can't create a fork of a repo inside the same organization.

Is that still the case? Based on github/roadmap#330 and https://github.blog/changelog/2022-06-27-improved-innersource-collaboration-and-enterprise-fork-policies/ I would have assumed this is now possible.

@vohoanglong0107
Copy link
Contributor Author

I'm not sure. Need someone with admin permission in rust-lang org to test this 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants