Re: [PATCH v3 0/3] Improve release notes process

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2017-01-10 at 11:23 -0500, John Ferlan wrote:
> I'm "mostly" fine with the changes - would like resolution sooner than
> later since I have a couple of things to add to news, but I didn't want
> to cause you extra merge cycles ;-)

Yeah, same here :)

> The *one* thing I'm really having a hard time with is not ending
> sentences with a period for <description>'s.

What about <summary>?

> Especially when there's
> two or more sentences within a <description> where the first one ends
> with a period, but the second one doesn't
> 
> Not ending the previous paragraph without a period was really difficult
> for my fingers.  I wouldn't hold up a ACK though for having or not
> having a period.

Patch 1/3 translates the current entries directly to the new
format, without any kind of editing.

As I've said before, I have zero problems having leading
periods for all sentences in the release notes, especially
if you feel so strongly about it :) We will have to change
them all, though: an inconsistent style just won't do.

So, will you ACK this series if I promise to follow up with
a patch that adds leading periods to all <description>s?

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux