Skip to content

Conversation

dowdiness
Copy link

This pull request fixes broken repository links on the Packages page.

Currently, links fail when a package's repository is defined with a Git-specific URL scheme (e.g., git+ssh:, git:, git+). This PR introduces logic to convert these schemes into a standard HTTPS format.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes broken package repository links by converting Git-specific URL schemes (git+ssh:, git+, git:) to standard HTTPS format, enabling proper navigation from the Packages page.

Key changes:

  • Added URL conversion logic to normalize Git repository URLs to HTTPS format

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

src/Packages.res Outdated
// Convert a repository url from SSH to HTTPS.
let repositoryHref = switch repositoryHref {
| href if String.startsWith(href, "git+ssh") => {
String.replace(href, "git+ssh:", "https:") }
Copy link

Copilot AI Oct 17, 2025

Choose a reason for hiding this comment

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

There is excessive trailing whitespace before the closing brace. This should be removed to maintain consistent formatting.

Copilot uses AI. Check for mistakes.

@fhammerschmidt
Copy link
Member

Sorry, the format check in CI was broken (fix here).
Please do a npm run format and this should be good to go.

@dowdiness
Copy link
Author

Thanks for the review! I've run npm run format, so the code should be formatted correctly now.

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