git: e22642d10e - main - committers-guide: pull request landing tweaks.

Warner Losh <[email protected]>
Newsgroups gmane.os.freebsd.devel.cvs.doc
Message-ID <[email protected]>
The branch main has been updated by imp:

URL: https://cgit.FreeBSD.org/doc/commit/?id=e22642d10ef896da9a08ac07a264f290b28b934c

commit e22642d10ef896da9a08ac07a264f290b28b934c
Author:     Warner Losh <[email protected]>
AuthorDate: 2021-06-03 13:55:34 +0000
Commit:     Warner Losh <[email protected]>
CommitDate: 2021-06-03 13:55:34 +0000

    committers-guide: pull request landing tweaks.
    
    Clarify that only closing the pull request resuires a GitHub
    account. All the other steps can proceed without one.
    
    Also, include --push-option=confirm-author in the push because pull
    requests will have a different author than the committer and this
    instructs the src pre-commit hooks to allow the commit through.
    
    Sponsored by:           Netflix
---
 documentation/content/en/articles/committers-guide/_index.adoc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/documentation/content/en/articles/committers-guide/_index.adoc b/documentation/content/en/articles/committers-guide/_index.adoc
index f3c63a4943..1767095908 100644
--- a/documentation/content/en/articles/committers-guide/_index.adoc
+++ b/documentation/content/en/articles/committers-guide/_index.adoc
@@ -2243,7 +2243,7 @@ When adjusting the message, add `Pull Request: https://github.com/freebsd-src/pu
 % git checkout main
 % git merge --ff-only staging
 <test again if needed>
-% git push freebsd
+% git push freebsd --push-option=confirm-author
 ....
 
 [.procedure]
@@ -2261,7 +2261,7 @@ For complicated pull requests that have multiple commits with conflicts, follow
 This will also work when bringing branches developed elsewhere into the local tree for committing.
 ====
 Once finished with the pull request, close it using GitHub's web interface.
-If the changes are fetched with https, this last step is the only one requiring a GitHub account.
+It is worth noting that if your `github` origin uses `https://`, the only step you'll need a GitHub account for is closing the pull request.
 
 [[vcs-history]]
 == Version Control History
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-doc-all
To unsubscribe, send any mail to "[email protected]"
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.