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
gc: recommend git gc --prune=now
instead of git prune
#652
Conversation
/submit |
Submitted as pull.652.git.1591581739031.gitgitgadget@gmail.com |
On the Git mailing list, Denton Liu wrote (reply to this):
|
`git prune` is a plumbing command and should not be run directly by users. The corresponding porcelain command is `git gc`, which is mentioned in the man page of `git prune`. Signed-off-by: John Lin <johnlinp@gmail.com>
83b7137
to
42aa638
Compare
Hi Denton,
gitgitgadget[bot] <notifications@github.com> 於 2020年6月8日 週一 下午6:19寫道:
On the Git mailing list, Denton Liu wrote (reply to this):
Hi John,
Thanks for working on this topic.
On Mon, Jun 08, 2020 at 02:02:18AM +0000, John Lin via GitGitGadget wrote:
> Subject: [PATCH] Recommend "git gc --prune=now" instead of "git prune"
The format for subjects is generally "<area>: <description>" so perhaps
something like "git gc: recommend use of `git gc --prune=now`"?
> From: John Lin ***@***.***>
>
> Signed-off-by: John Lin ***@***.***>
This commit is missing some justification. In the commit message, we
should mention that `git prune` is a plumbing command and in the
documentation for it, it mentions that the porcelain `git gc` command
should be used instead.
Thank you for the suggestions. I've updated the commit message. Please
take a look again. Thanks.
Best,
John Lin
…
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
/submit |
Submitted as pull.652.v2.git.1591662224566.gitgitgadget@gmail.com |
git gc --prune=now
instead of git prune
Thanks for the update, John. By the way, when responding to the emails from the mailing list, you should respond to the actual emails that you get CC'd on. If you respond to the GGG emails from GitHub, the response is only posted here and not everyone checks GitHub regularly. |
Ouch. I didn't notice this. Thanks for telling me. I'll reply to the actual email next time. |
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, 林自均 wrote (reply to this):
|
On the Git mailing list, Denton Liu wrote (reply to this):
|
On the Git mailing list, 林自均 wrote (reply to this):
|
Please note that closing PRs and opening new ones loses all history, and also does not add the range-diff that is so helpful for reviewers. Next time, just force-push to the same PR and |
@dscho Thank you for the advice. I would like to ask more about this issue. I understand that I shouldn't close #651 and create #652 since the 2 PRs are closed related and the history of #651 is important for the reviewers in #652. However, I think it's clearer to create a new PR for the bash completion of |
Of course, you can do what you want, but if I had submitted these patches, I would have made them v1 and v2. |
I see. Thank you for the suggestions. |
git prune
is a plumbing command and should not be run directly byusers. The corresponding porcelain command is
git gc
, which ismentioned in the man page of
git prune
.Signed-off-by: John Lin johnlinp@gmail.com
Fix according to #642.