People who are willing to review documentation changes (opt-in)
Details
Today
Get rid of .Xr in .Nd and make the .Nd itself more informative.
Thank you for the review!
Yesterday
I have not reviewed this because all I know about colors is how to turn them off, drives me crazy. Maybe... @dteske might know?
That is not necessary.
- We do not really have tooling around that, so it does not make it any more discoverable.
- Whoever lands it is responsible for handling the judgement. So, jilles approval stands unless he rescinds it or a change is made which we would deem requires his re-review per the context.
- Someone might do that as a form of ping, since it sends out an email.
For example, since jilles is reviewing it in the context of being the lead shell maintainer, we could still make mechanical changes at this stage like fixing a typo or improving text alignment. However a content change that changes the meaning would, to me, invalidate his previous approval and require his re-review.
Thank you both, I think this is a great new feature for the sh manual and I am so excited to land this now!
@jilles Could you please accept this revision once again, just to move it into 'ready to land' category?
Ping.
Sun, Jul 19
Rebase the patch and bump .Dd.
Sat, Jul 18
Ping.
Ping.
Ping.
Tue, Jul 7
Mon, Jul 6
@ziaee Could you please MFC this commit (I see the commit message says it is to be MFCed after 3 days, but it's still not).
Sun, Jul 5
Yes, as you said, change it.
https://docs.asciidoctor.org/asciidoc/latest/sections/bibliography/
Removing the hyphen in the first line is fine, but then that should be mentioned in the commit message. I can add a sentence to that effect myself. Which mail do you want to use? The one that can be inferred from your Phabricator login? I also see a mail at "zohomail" in the tree that could match. If you have to disclose the full name and don't want to do that here, then please mail it to me (olce@).
Sat, Jul 4
Remove hyphen from the first comment line. Thanks @ziaee!
Ping for review.
Fri, Jul 3
Thu, Jul 2
Do you see any problems with them?
We can't make changes to past release notes once they are published. They become part of the release hash and would change in a number of places that reference them (even if they contain typos).
