Re: [libvirt] [PATCH 3/2] docs: Document adaptive timeout for qemu monitor

On Wed, Mar 15, 2017 at 01:06:12PM +0100, Michal Privoznik wrote: s/seconds/second/ (as in 30-second timeout) feels more English, I guess. I'll take the opportunity to repeat here what I said in another news-related thread. The good thing about the new news file layout is that we can express freely what is the feature that was added and we don't have to copy-paste the restricted commit messages. So I, personally, like to have nicely formatted sentences in the summary (well, I like that even in the commit messages sometimes, but that's another story). I would go with: "When connecting to qemu monitor, the timeout is now adaptive" or even: "Better (or dynamic determination) strategy is now used for qemu monitor connection timeout" for the summary. Or something along the lines. What I say is (as before) pretty subjective, so I'll leave the final decision up to you, just wanted to put it out there (yet again). I'm trying to imagine the user going through these and immediately having an idea of the list of things being done. Commit names are more for developers. ACK P.S.: Isn't it nice when you can just copy-paste reviews? =D P.P.S.: Would it be to much if I also said that shortening the description to something like the following would be enough? "When launching QEMU, libvirt now adjusts the timeout for the initial connection dynamically as oposed to the static 30 seconds before" Still, your decision on that.