Ticket #4316 enhancement closed fixed

Opened 3 years ago

Last modified 3 years ago

Changing versions in NEWS files for pre-release increments is too hard

Reported by: jml Owned by:
Priority: normal Milestone: regular-releases
Component: release management Keywords: review
Cc: Branch: branches/drop-pre-release-4316
Author: jml Launchpad Bug:

Description

change-versions doesn't change the version in NEWS files. build-news adds a new heading on top of the previous pre-release heading, e.g.

Twisted Core 10.0.0pre2 (2010-02-24)
====================================

Twisted Core 10.0.0pre1 (2010-02-21)
====================================

Ideally there'd be some way to bump the pre-release version. Perhaps change-versions should do this?

Change History

1

Changed 3 years ago by jml

  • milestone set to regular-releases

2

Changed 3 years ago by jml

  • branch set to branches/drop-pre-release-4316
  • branch_author set to jml

(In [29225]) Branching to 'drop-pre-release-4316'

3

Changed 3 years ago by jml

  • owner radix deleted
  • keywords review added

In this branch, change-versions will alter the version and date in all NEWS files if the current release is a pre-release.

4

Changed 3 years ago by TimAllen

  • owner set to TimAllen

5

Changed 3 years ago by TimAllen

  • keywords review removed
  • owner changed from TimAllen to jml

My only change would be:

--- a/twisted/python/_release.py
+++ b/twisted/python/_release.py
@@ -143,7 +143,7 @@ def changeAllProjectVersions(root, versionTemplate, today=None):
                              versionTemplate.minor, versionTemplate.micro,
                              prerelease=versionTemplate.prerelease)
 
-        if project.getVersion().prerelease:
+        if oldVersion.prerelease:
             builder = NewsBuilder()
             builder._changeNewsVersion(
                 root.child("NEWS"), builder._getNewsName(project),

...since every other call to project.getVersion() in this function has been replaced by oldVersion.

With that change, +1 for merging from me.

6

Changed 3 years ago by jml

  • keywords review added
  • owner jml deleted

Done & thanks for the review. I'm going to hold off on landing this branch until a core dev reviews it, to satisfy my own paranoia.

7

Changed 3 years ago by jml

  • status changed from new to closed
  • resolution set to fixed

(In [29242]) Merge drop-pre-release-4316

  • Author: jml
  • Reviewer: TimAllen
  • Fixes #4316

When running change-version on a pre-release version of Twisted, change the references to the pre-release version in NEWS entries.

8

Changed 2 years ago by <automation>

Note: See TracTickets for help on using tickets.