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

cargo: point repository metadata to clonable URLs #68

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

lucab
Copy link
Contributor

@lucab lucab commented Jun 10, 2024

This tweaks the repository fields in Cargo metadata in order to use the correct (i.e. git clonable) URL. The existing GitHub webUI URLs for each package have been retained and moved to homepage fields.

@lucab
Copy link
Contributor Author

lucab commented Jun 10, 2024

Same rationale as RustCrypto/elliptic-curves#1055.

@lucab
Copy link
Contributor Author

lucab commented Jun 11, 2024

CI is red due to a pre-existing failure, I've opened #69 to fix that.

This tweaks the `repository` fields in Cargo metadata in order to use the correct (i.e. git clonable) URL.
The existing GitHub webUI URLs for each package have been retained and moved to `homepage` fields.
@lucab lucab force-pushed the ups/cargo-repo-url branch from 5388bff to 4bd6a8c Compare June 11, 2024 16:35
@lucab
Copy link
Contributor Author

lucab commented Jun 11, 2024

Rebased, CI is green now.

@tarcieri tarcieri merged commit 6d5eae7 into RustCrypto:master Jun 11, 2024
83 checks passed
@lucab lucab deleted the ups/cargo-repo-url branch June 12, 2024 06:23
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.

2 participants