BTW, could you try to submit future patches with context? The easiest way is to configure arcanist (https://www.freshports.org/devel/arcanist/). It makes reviewing a bit easier. Thank you :)
Hmmm, I think that this file should be sorted by surname.
Technically, portmgr sent the email on Jan, 2. ;)
Unnecessary space after <p>?
Seems fine. Please, before you commit,
- check your XML changes with igor (see https://www.freebsd.org/doc/en_US.ISO8859-1/books/fdp-primer/book.html for more info).
- make sure that your git setup follows the recommendations from this repo: https://github.com/bsdimp/freebsd-git-docs
- before you push, take a look at git log if everything is fine (not only the patch itself but also the metadata like author name and email)
BTW, I'd like to ask you to add the commit message you are going to submit in the description of the patch here on Phabricator. This way I'll be able to review the contents of the commit message and introduce you to some conventions if necessary.
Almost there... :D
The commit message I want to use is:
doc: Add committer entries for Neel Chauhan <nc> Steps 1-4 from the Committer's guide: - Add entries for Neel Chauhan <nc> - Add GPG key Reviewed by: 0mp (mentor) Approved by: 0mp (mentor) Differential Revision: https://reviews.freebsd.org/D27957
Is this okay?
I;ll use this:
Add committer entries for Neel Chauhan <nc> Steps 1-4 from the Committer's guide: - Add entries for Neel Chauhan <nc> - Add GPG key Reviewed by: 0mp Approved by: 0mp Differential Revision: https://reviews.freebsd.org/D27957