Re: [Xen-devel] [PATCH] Introduce CHANGELOG.md
> On 10 Jan 2020, at 17:54, Andrew Cooper wrote: > > On 10/01/2020 09:12, Paul Durrant wrote: >> As agreed during the 2020-01 community call [1] this patch introduces a >> changelog, based on the principles explained at keepachangelog.com [2]. >> A new MAINTAINERS entry is also added, with myself as (currently sole) >> maintainer. >> >> [1] See C.2 at https://cryptpad.fr/pad/#/2/pad/edit/ERZtMYD5j6k0sv-NG6Htl-AJ/ >> [2] https://keepachangelog.com/en/1.0.0/ >> >> Signed-off-by: Paul Durrant >> --- >> Cc: Andrew Cooper >> Cc: George Dunlap >> Cc: Ian Jackson >> Cc: Jan Beulich >> Cc: Julien Grall >> Cc: Konrad Rzeszutek Wilk >> Cc: Stefano Stabellini >> Cc: Wei Liu >> Cc: Lars Kurth >> >> Should there be other maintainers apart from myself (with my RM hat on)? >> Perhaps Lars should also be added as a designated reviewer? > > Ultimately, the committers are last line of judgement on "whether this > change should be in the changelog". Practically, that includes "The > Rest", but there was an objection to that on the call IIRC. Am happy to be added Lars ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce CHANGELOG.md
On 10/01/2020 09:12, Paul Durrant wrote: > As agreed during the 2020-01 community call [1] this patch introduces a > changelog, based on the principles explained at keepachangelog.com [2]. > A new MAINTAINERS entry is also added, with myself as (currently sole) > maintainer. > > [1] See C.2 at https://cryptpad.fr/pad/#/2/pad/edit/ERZtMYD5j6k0sv-NG6Htl-AJ/ > [2] https://keepachangelog.com/en/1.0.0/ > > Signed-off-by: Paul Durrant > --- > Cc: Andrew Cooper > Cc: George Dunlap > Cc: Ian Jackson > Cc: Jan Beulich > Cc: Julien Grall > Cc: Konrad Rzeszutek Wilk > Cc: Stefano Stabellini > Cc: Wei Liu > Cc: Lars Kurth > > Should there be other maintainers apart from myself (with my RM hat on)? > Perhaps Lars should also be added as a designated reviewer? Ultimately, the committers are last line of judgement on "whether this change should be in the changelog". Practically, that includes "The Rest", but there was an objection to that on the call IIRC. > --- > CHANGELOG.md | 14 ++ > MAINTAINERS | 5 + > 2 files changed, 19 insertions(+) > create mode 100644 CHANGELOG.md > > diff --git a/CHANGELOG.md b/CHANGELOG.md > new file mode 100644 > index 00..ec5e174aa0 > --- /dev/null > +++ b/CHANGELOG.md > @@ -0,0 +1,14 @@ > +# Changelog > + > +All notable changes to Xen will be documented in this file. > + > +The format is based on [Keep a > Changelog](https://keepachangelog.com/en/1.0.0/) > + > +## [Unreleased](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog) Has anyone looked over staging to find other noteworthy things? ~Andrew > + > +### Added > + - This file and MAINTAINERS entry. > + > +## > [4.13.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.13.0) > - 2019-12-17 > + > +> Pointer to release from which CHANGELOG tracking starts > diff --git a/MAINTAINERS b/MAINTAINERS > index d5bd83073c..68c691361a 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -198,6 +198,11 @@ F: xen/include/asm-arm/ > F: xen/include/public/arch-arm/ > F: xen/include/public/arch-arm.h > > +Change Log > +M: Paul Durrant > +S: Maintained > +F: CHANGELOG.md > + > Continuous Integration (CI) > M: Doug Goldstein > W: https://gitlab.com/xen-project/xen ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce CHANGELOG.md
> -Original Message- > From: Jan Beulich > Sent: 10 January 2020 09:52 > To: Durrant, Paul > Cc: xen-devel@lists.xenproject.org; Andrew Cooper > ; George Dunlap ; > Ian Jackson ; Julien Grall ; > Konrad Rzeszutek Wilk ; Stefano Stabellini > ; Wei Liu ; Lars Kurth > > Subject: Re: [PATCH] Introduce CHANGELOG.md > > On 10.01.2020 10:12, Paul Durrant wrote: > > --- /dev/null > > +++ b/CHANGELOG.md > > @@ -0,0 +1,14 @@ > > +# Changelog > > + > > +All notable changes to Xen will be documented in this file. > > How do we qualify what's "notable" and what's not? IOW I wonder > whether "All" should be dropped, or be replaced by "Some". > Agreed that it's debatable. Perhaps just drop the 'All' and say: 'Notable changes to Xen will be documented in this file.' ? Patch authors ought to update the file if they consider their contribution(s) notable but I'd also hope that maintainers will express an opinion as to whether something should be included/not included. It's not going to be fool-proof but I think it will be better than nothing. Paul > Jan ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce CHANGELOG.md
On 10/01/2020, 09:12, "Paul Durrant" wrote: As agreed during the 2020-01 community call [1] this patch introduces a changelog, based on the principles explained at keepachangelog.com [2]. A new MAINTAINERS entry is also added, with myself as (currently sole) maintainer. [1] See C.2 at https://cryptpad.fr/pad/#/2/pad/edit/ERZtMYD5j6k0sv-NG6Htl-AJ/ [2] https://keepachangelog.com/en/1.0.0/ Signed-off-by: Paul Durrant Thank you Paul Exactly what I was looking for Reviewed-by: Lars Kurth ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce CHANGELOG.md
On 10.01.2020 10:12, Paul Durrant wrote: > --- /dev/null > +++ b/CHANGELOG.md > @@ -0,0 +1,14 @@ > +# Changelog > + > +All notable changes to Xen will be documented in this file. How do we qualify what's "notable" and what's not? IOW I wonder whether "All" should be dropped, or be replaced by "Some". Jan ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
[Xen-devel] [PATCH] Introduce CHANGELOG.md
As agreed during the 2020-01 community call [1] this patch introduces a changelog, based on the principles explained at keepachangelog.com [2]. A new MAINTAINERS entry is also added, with myself as (currently sole) maintainer. [1] See C.2 at https://cryptpad.fr/pad/#/2/pad/edit/ERZtMYD5j6k0sv-NG6Htl-AJ/ [2] https://keepachangelog.com/en/1.0.0/ Signed-off-by: Paul Durrant --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Julien Grall Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Wei Liu Cc: Lars Kurth Should there be other maintainers apart from myself (with my RM hat on)? Perhaps Lars should also be added as a designated reviewer? --- CHANGELOG.md | 14 ++ MAINTAINERS | 5 + 2 files changed, 19 insertions(+) create mode 100644 CHANGELOG.md diff --git a/CHANGELOG.md b/CHANGELOG.md new file mode 100644 index 00..ec5e174aa0 --- /dev/null +++ b/CHANGELOG.md @@ -0,0 +1,14 @@ +# Changelog + +All notable changes to Xen will be documented in this file. + +The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) + +## [Unreleased](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog) + +### Added + - This file and MAINTAINERS entry. + +## [4.13.0](https://xenbits.xen.org/gitweb/?p=xen.git;a=shortlog;h=RELEASE-4.13.0) - 2019-12-17 + +> Pointer to release from which CHANGELOG tracking starts diff --git a/MAINTAINERS b/MAINTAINERS index d5bd83073c..68c691361a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -198,6 +198,11 @@ F: xen/include/asm-arm/ F: xen/include/public/arch-arm/ F: xen/include/public/arch-arm.h +Change Log +M: Paul Durrant +S: Maintained +F: CHANGELOG.md + Continuous Integration (CI) M: Doug Goldstein W: https://gitlab.com/xen-project/xen -- 2.17.1 ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel