Skip to content

fix(cli): support mixed-case GitHub hostnames in PR command#231

Open
mvmax-dev wants to merge 1 commit into
CortexLM:mainfrom
mvmax-dev:fix/github-url-parsing-mixed-case
Open

fix(cli): support mixed-case GitHub hostnames in PR command#231
mvmax-dev wants to merge 1 commit into
CortexLM:mainfrom
mvmax-dev:fix/github-url-parsing-mixed-case

Conversation

@mvmax-dev
Copy link
Copy Markdown

This PR fixes a bug where fails to parse GitHub remote URLs containing mixed-case hostnames (e.g., ).

Changes:

  • Updated to perform case-insensitive hostname detection.
  • Preserves case for owner and repository names.
  • Added test cases for mixed-case HTTPS and SSH URLs.

Resolves PlatformNetwork/bounty-challenge#53520

Wallet for attribution: 0x25f58A305A5095fb8Eb84b422a14d705A8dfa278

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

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.

[BUG] [v0.0.7] cortex pr fails to parse GitHub remote URL with mixed-case host, resulting in incorrect repository detection

1 participant