镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Rugged::Repository.clone_at checkout_branch option - #339

Merged
arthurschreiber merged 3 commits into
libgit2:developmentfrom
albertsun:repository-clone-at-checkout-branch
Mar 25, 2014
Merged

arthurschreiber merged 3 commits into
libgit2:developmentfrom
albertsun:repository-clone-at-checkout-branch

Conversation

@albertsun

Copy link
Copy Markdown
Contributor

Picks up where #338 left off.

Comment thread test/online/clone_test.rb Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is a good idea, but does not work. The credentials returned by ssh_key_credential don't work with the repo at libgit2/TestGitRepository.

Fixing this test case will involve a lot of changes to make use of the locally served repository, and it's not really in scope for this Pull Request. CAn you just remove this test for now so I can get your changes merged without breaking the build?

@arthurschreiber

Copy link
Copy Markdown
Member

Sorry for all these nitpicks. If you can incorporate these changes. this PR will be good to be merged. 😄

…ravis build. Added assertion to test_clone_with_branch
@albertsun

Copy link
Copy Markdown
Contributor Author

Yep. All set now I think.

arthurschreiber added a commit that referenced this pull request Mar 25, 2014
…ranch

Rugged::Repository.clone_at checkout_branch option
@arthurschreiber
arthurschreiber merged commit b73e523 into libgit2:development Mar 25, 2014
@arthurschreiber

Copy link
Copy Markdown
Member

Thanks a lot!

@albertsun
albertsun deleted the repository-clone-at-checkout-branch branch March 25, 2014 18:34
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