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

Bind git_submodule_repo_init #914

Merged
merged 1 commit into from
Jan 19, 2023
Merged

Bind git_submodule_repo_init #914

merged 1 commit into from
Jan 19, 2023

Conversation

Hyask
Copy link

@Hyask Hyask commented Jan 18, 2023

This allows a real initialization of a submodule's repository, where git_submodule_init only leaves you with nothing truly usable.

Hyask pushed a commit to tetrane/checkout that referenced this pull request Jan 18, 2023
…tory

Following rust-lang/git2-rs#914, we can now use
`repo_init`, which gives us a real repository that we can clone to populate it.
@ehuss
Copy link
Contributor

ehuss commented Jan 18, 2023

Thanks, can you add a test for it?

@Hyask
Copy link
Author

Hyask commented Jan 18, 2023

Thanks, can you add a test for it?

I thought about it, but after a quick look at the end of submodule.rs, I wasn't really sure where to start. Are there the only submodule tests? I'll try to add something based on existing clone_submodule() test.

@ehuss
Copy link
Contributor

ehuss commented Jan 18, 2023

Yea, I think it would go at the bottom of submodule.rs. And yea, I would just start be copying one of the other tests and then adjusting it to utilize the new method.

@Hyask
Copy link
Author

Hyask commented Jan 19, 2023

I've added a test, do you think it's enough, or is more needed?

Copy link
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@ehuss ehuss merged commit c046b19 into rust-lang:master Jan 19, 2023
@Hyask
Copy link
Author

Hyask commented Jan 20, 2023

@ehuss thanks for the quick review and the merge. Do you have any idea when this will be released in a tag?

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