Skip to content

Conversation

@Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Nov 14, 2025

This PR implements the majority of the frontend work for supporting GitLab in our Trusted Publishing flows. Specifically, it adjusts the crate settings section to list both GitHub and GitLab configurations, it allows crate owners to delete GitLab configurations, and it allows them to create new configurations for GitLab too.

Related

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works and removed C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Nov 14, 2025
@Turbo87 Turbo87 requested a review from a team November 14, 2025 10:07
@Turbo87 Turbo87 moved this to For next meeting in crates.io team meetings Nov 14, 2025
Copy link
Contributor

@LawnGnome LawnGnome left a comment

Choose a reason for hiding this comment

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

One messaging question; otherwise LGTM.


<div class='note'>
crates.io currently only supports GitHub, but we are planning to support other platforms in the future.
Select the CI/CD platform where your publishing workflow is configured.
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't know if we want to continue to hint that we may support other platforms in the future here. I'm fine with this as it is, but just flagging that we're removing that from here.

Copy link
Member Author

Choose a reason for hiding this comment

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

while supporting more platforms would be good, I don't think I can justify spending more time on adding additional ones, so that implementation would have to be done by the community.

since the previous statement said "planning to support other platforms in the future" I think it's probably fine to remove that part now that we do support an additional platform.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-frontend 🐹 C-enhancement ✨ Category: Adding new behavior or a change to the way an existing feature works

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants