Replies: 6 comments 3 replies
-
I think it may be a little early to prioritise this. The more hosting providers we support early, the larger the changes we need to make when touching that part of the system and with the package registry potentially being a thing soon™️ this area of the system may go through changes. I would definitely like to get to this one day, but it feels like it's a low priority for now. How do you feel about it @finestructure? |
Beta Was this translation helpful? Give feedback.
-
Oh I definitely agree there Dave, this is low priority. I was more asking for my own understanding and so that we have it written down. While it doesn't necessarily change our approach to the way we develop things, it doesn't hurt to have in the back of your mind approaches that could be avoided or done in such a strategic way that doesn't make this incredibly difficult in the future! |
Beta Was this translation helpful? Give feedback.
-
Tagging @Mordil here as he brought this up on Twitter recently and may potentially want to work on GitLab support. Before it gets to PR, it's probably worth a quick discussion here of the approach with @finestructure involved. |
Beta Was this translation helpful? Give feedback.
-
Whoops, I never actually replied to this thread it seems - apologies! Just in terms of procedure in case someone were to tackle this: we've currently got three dependencies reaching out to Github (see var fetchLicense: (_ client: Client, _ package: Package) -> EventLoopFuture<Github.License?>
var fetchMetadata: (_ client: Client, _ package: Package) -> EventLoopFuture<Github.Metadata>
var fetchReadme: (_ client: Client, _ package: Package) -> EventLoopFuture<Github.Readme?> and they're all used in It's not a whole lot but it would obviously have to be mapped to the repsective vendor APIs. I know Gitlab has a GraphQL API, so that should make it easier for that particular vendor. We're also already talking to the Gitlab API for the build system but we'd need to be careful with respect to API keys and rate limits to ensure the two systems don't rate limit each other. (Although that's going to be mitigated by a lot fewer packages on Gitlab and on Github.) None of this is terribly complicated but it's a fair amount of leg work to be done. In addition, there's a bit of operational risk if there were lots of Gitlab packages and API limits. There's also the maintenance aspect to be kept in mind, both when it comes to additional properties we're pulling in and that need to be added across multiple providers as well as operationally when errors happen. (And Gitlab does have a bit of a history with instability...) I think it'd be really important to get a feel for how many packages live on other hosting providers before embarking on this. And while I'm not a fan to have just a single provider, it would certainly make our lives easier if mirroring to Github was an option if it's just a few. |
Beta Was this translation helpful? Give feedback.
-
I'm going to rename this to "Add support for GitLab". That's a concrete thing we want to do and that's the most likely source of a new package right now, I know @Mordil has some packages on GitLab that he'd like to add. Concrete tasks are more likely to get implemented, and we can add an issue for "Add BitBucket support" after this is closed. |
Beta Was this translation helpful? Give feedback.
-
What's the high level of effort on this these days, now that a bunch of (awesome) features have been added to the Index? It's becoming a question again after considering dropping the GitHub mirror for my GitLab packages: https://forums.swift.org/t/redistack-and-repository-mirroring/56518/10 |
Beta Was this translation helpful? Give feedback.
-
Add support for URLs in the PackageList to come from GitLab as well as GitHub.
Original issue text:
Beta Was this translation helpful? Give feedback.
All reactions