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

Delete empty gitmodules #789

Merged
merged 8 commits into from Nov 3, 2019
Merged

Conversation

go2null
Copy link
Contributor

@go2null go2null commented Oct 20, 2019

  • Handle case where this is the first submodule, so it leaves the git repository as it was before.
  • Handle case where this is the last submodule.
  • Check before declaring victory.
  • Code reformatting and re-organization.
  • Added option to continue on error to allow cleaning up failed deletions.

NOTE: Not validated on Mac OS X and BSD.

bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
@go2null go2null force-pushed the delete-empty-gitmodules branch 2 times, most recently from 2bc8690 to cff057c Compare October 23, 2019 14:45
@go2null
Copy link
Contributor Author

go2null commented Oct 24, 2019

BTW, here is the script I used to test. Not sure where to save it, attaching it here.
test-git-delete-submodule.txt

bin/git-delete-submodule Outdated Show resolved Hide resolved
bin/git-delete-submodule Outdated Show resolved Hide resolved
@spacewander spacewander merged commit f2934bd into tj:master Nov 3, 2019
@spacewander
Copy link
Collaborator

Merged. Thanks!

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.

None yet

2 participants