New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PATCH] 821176d For changelogs suggest name it Changes and follow CPAN::Changes::Spec #13117
Comments
From @neilbThis is a small doc patch for permodstyle relating to changelogs. Cheers, |
From @neilb0001-For-changelogs-suggest-name-it-Changes-and-follow-CP.patchFrom 821176ddd17eb8387c5a695125b66ff40c316e68 Mon Sep 17 00:00:00 2001
From: Neil Bowers <neil@bowers.com>
Date: Sun, 21 Jul 2013 20:59:20 +0100
Subject: [PATCH] For changelogs suggest name it Changes and follow
CPAN::Changes::Spec
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="------------1.7.6"
This is a multi-part message in MIME format.
--------------1.7.6
Content-Type: text/plain; charset=UTF-8; format=fixed
Content-Transfer-Encoding: 8bit
---
pod/perlmodstyle.pod | 5 +++++
t/porting/known_pod_issues.dat | 1 +
2 files changed, 6 insertions(+), 0 deletions(-)
--------------1.7.6
Content-Type: text/x-patch; name="0001-For-changelogs-suggest-name-it-Changes-and-follow-CP.patch"
Content-Transfer-Encoding: 8bit
Content-Disposition: attachment; filename="0001-For-changelogs-suggest-name-it-Changes-and-follow-CP.patch"
diff --git a/pod/perlmodstyle.pod b/pod/perlmodstyle.pod
index 558642c..5622d32 100644
--- a/pod/perlmodstyle.pod
+++ b/pod/perlmodstyle.pod
@@ -598,6 +598,11 @@ Release notes or changelogs should be produced for each release of your
software describing user-visible changes to your module, in terms
relevant to the user.
+Unless you have good reasons for using some other format
+(for example, a format used within your company),
+the convention is to name your changelog file C<Changes>,
+and to follow the simple format described in L<CPAN::Changes::Spec>.
+
=head1 RELEASE CONSIDERATIONS
=head2 Version numbering
diff --git a/t/porting/known_pod_issues.dat b/t/porting/known_pod_issues.dat
index cdd5971..2f9e7d0 100644
--- a/t/porting/known_pod_issues.dat
+++ b/t/porting/known_pod_issues.dat
@@ -36,6 +36,7 @@ Classic::Perl
Clone
cpan2dist(1)
cpanp(1)
+CPAN::Changes::Spec
CPANPLUS
Crypt::Random
curl(1)
--------------1.7.6--
|
From @jkeenanOn Sun Jul 21 13:30:19 2013, neil.bowers@cogendo.com wrote:
I hesitate to +1 this because CPAN::Changes::Spec is not distributed $ corelist CPAN::Changes::Spec |
The RT System itself - Status changed from 'new' to 'open' |
From @rurbanOn Sun, Jul 21, 2013 at 7:10 PM, James E Keenan via RT
LGTM
He explained why he added this name to the list above. It has nothing
-- |
From @karenetheridgeOn Mon, Jul 22, 2013 at 10:33:17AM -0500, Reini Urban wrote:
If the concern is "we shouldn't point to specs that we haven't officially |
From @rurbanOn Mon, Jul 22, 2013 at 2:24 PM, Karen Etheridge <perl@froods.org> wrote:
meh, not necessary None of the JSON, YAML or CPAN spec suites are in core, and don't need to be. -- |
From @neilb
perlmodstyle starts with the line: This document attempts to describe the Perl Community's "best practice" I think CPAN::Changes::Spec is the community's best practice, given 40% of dists follow it, and there isn't a competing spec, that I'm aware of. I don't think modules should have to be included in core for them to be mentioned in core doc. Neil |
From @rjbs* Neil Bowers <neil.bowers@cogendo.com> [2013-07-22T16:53:41]
Thanks, I was hoping to find out that number. I think appyling this is reasonable. -- |
From @tonycozOn Sun Jul 21 13:30:19 2013, neil.bowers@cogendo.com wrote:
Thanks, applied as 4a8dd74. Tony |
@tonycoz - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#118959 (status was 'resolved')
Searchable as RT118959$
The text was updated successfully, but these errors were encountered: