Skip to content

Conversation

@l0ngvh
Copy link
Contributor

@l0ngvh l0ngvh commented Dec 8, 2024

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)

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

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.

@l0ngvh
Copy link
Contributor Author

l0ngvh commented Dec 9, 2024

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

@Kobzol
Copy link
Member

Kobzol commented May 23, 2025

As per #182 (comment), we shouldn't need this anymore. Thank you for working on this though!

@Kobzol Kobzol closed this May 23, 2025
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