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

Add submodules clone checkbox #1188

Merged
merged 3 commits into from
Nov 28, 2022
Merged

Add submodules clone checkbox #1188

merged 3 commits into from
Nov 28, 2022

Conversation

seignovert
Copy link
Contributor

@seignovert seignovert commented Nov 25, 2022

Resolves #1187

  • Add an optional submodules bolean argument to Git.clone() to append --recurse-submodules flag (default False)
  • Add test_git_submodules_success test
  • Add data["submodules"] to GitCloneHandler
  • Add a new submodules checkbox in GitCloneForm (checked by default)
  • Renamed checkbox input to download in GitCloneForm

The clone dialog looks like this:

image


Edited to add a new clone dialog image

@welcome
Copy link

welcome bot commented Nov 25, 2022

Thanks for submitting your first pull request! You are awesome! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please make sure you followed the pull request template, as this will help us review your contribution more quickly.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also a intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@github-actions
Copy link

Binder 👈 Launch a binder notebook on branch seignovert/jupyterlab-git/master

Copy link
Member

@fcollonval fcollonval left a comment

Choose a reason for hiding this comment

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

Thanks a lot @seignovert

This is awesome with doc, test and new UI 🚀

@fcollonval fcollonval merged commit 69c6122 into jupyterlab:master Nov 28, 2022
@welcome
Copy link

welcome bot commented Nov 28, 2022

Congrats on your first merged pull request in this project! 🎉
congrats
Thank you for contributing, we are very proud of you! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clone git submodules
2 participants