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

Some documentation cleanups #4

Merged
merged 1 commit into from
Aug 13, 2019
Merged

Some documentation cleanups #4

merged 1 commit into from
Aug 13, 2019

Conversation

ethomson
Copy link
Contributor

@ethomson ethomson commented Aug 1, 2019

  • Only specify -f (force) once.
  • Submodule documentation improvements

@ethomson
Copy link
Contributor Author

ethomson commented Aug 1, 2019

Note: I'm not sure about the correctness of my claims about submodules and their defaults. 😀

@kivikakk
Copy link

I did some empirical testing (because I wanted submodules!):

The difference is, with submodules: true the following get executed at the end of the action:

git submodule sync
git -c http.https://github.com.extraheader="AUTHORIZATION: basic ***" submodule update --init --force

So I'd say the proposed improvement is accurate!

@TingluoHuang
Copy link
Member

@ethomson we do run git clean -ffdx to delete nested git repo under the main repo.

If the Git configuration variable clean.requireForce is not set to false, git clean will refuse to delete files or directories unless given -f, -n or -i. Git will refuse to delete directories with .git sub directory or file unless a second -f is given.

@ethomson
Copy link
Contributor Author

Thanks @TingluoHuang, I checked the docs again and you're obviously right.

@ethomson ethomson force-pushed the users/ethomson/docs branch from fa0a7ee to 04a5459 Compare August 13, 2019 13:19
@TingluoHuang TingluoHuang merged commit 53bed07 into master Aug 13, 2019
@TingluoHuang TingluoHuang deleted the users/ethomson/docs branch August 13, 2019 19:44
srini-hv pushed a commit to srini-hv/checkout that referenced this pull request May 19, 2021
srini-hv pushed a commit to srini-hv/checkout that referenced this pull request May 20, 2021
srini-hv pushed a commit to srini-hv/checkout that referenced this pull request May 20, 2021
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

3 participants