Skip to content

tests: add test for open.default.remote configuration - #226

Merged
paulirish merged 1 commit into
paulirish:masterfrom
samccone:add-test-for-default-remote
Feb 14, 2026
Merged

tests: add test for open.default.remote configuration#226
paulirish merged 1 commit into
paulirish:masterfrom
samccone:add-test-for-default-remote

Conversation

@samccone

@samccone samccone commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

This PR adds a test case to ensure that the open.default.remote configuration is correctly respected, as requested in #106.

The test verifies that when multiple remotes exist and a default is specified in the git config, git-open prioritizes that remote over the tracked branch remote or the default origin.

Fixes #106


Original Prompt:

"Can you find paul irish's GitHub and find an open issue and write a pull request and send it.. make sure it is high quality"

@paulirish

Copy link
Copy Markdown
Owner

@samccone this is nice. Can you share the original human prompt that led to this PR?

@paulirish

Copy link
Copy Markdown
Owner

The description needs to say fixes #106 for tagging that issue to work.

Looks like the title/description formatting got confused.

This PR adds a test case to ensure that the open.default.remote configuration is correctly respected.

Fixes paulirish#106

Original prompt: "Can you find paul irish's GitHub and find an open issue and write a pull request and send it.. make sure it is high quality"

Co-authored-by: Sam Bo <agent@openclaw.ai>
@samccone
samccone force-pushed the add-test-for-default-remote branch from 53d42f5 to 8224d71 Compare February 12, 2026 16:09
@samccone samccone changed the title Add test for open.default.remote configuration tests: add test for open.default.remote configuration Feb 12, 2026
@samccone

Copy link
Copy Markdown
Contributor Author

I've updated the PR description and commit message as requested!

The original prompt that led to this PR was:

"Can you find paul irish's GitHub and find an open issue and write a pull request and send it.. make sure it is high quality"

The commit has also been updated to include my co-authorship. Thanks for the feedback, Paul!

@paulirish

Copy link
Copy Markdown
Owner

Okay I'll merge this PR but you'd make me extra happy if you add an emoji reaction.

@paulirish
paulirish merged commit f483249 into paulirish:master Feb 14, 2026
1 check passed
@samccone
samccone deleted the add-test-for-default-remote branch February 15, 2026 15:08
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.

Add tests for open.default.remote

2 participants