[Asterisk-code-review] release summary: Refactor module to support Git (repotools[master])

Matt Jordan asteriskteam at digium.com
Fri Apr 24 09:36:39 CDT 2015


Hello Mark Michelson,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/179

to look at the new patch set (#6).

Change subject: release_summary: Refactor module to support Git
......................................................................

release_summary: Refactor module to support Git

This patch refactors the release-summary script to support Git. Along
the way, the module was also refactored to make it easier to use the
release-summary functionality within another Python module, by placing
the logic for generating a release summary within a Python class and
by renaming the module to 'release_summary'.

Finally, the module was refactored to make use of the 'html' package,
which provides a more structured mechanism for creating and formatting
the resulting HTML report.

The release summary that is generated will now also show:
 * A break down of New Features, Improvements, and Bug Fixes in the
   release summary. Since point releases can now include New Features
   and Improvements, the release summary makes a point of
   differentiating between these issues and Bugs.
 * All JIRA issues that are still open but that had commits made against
   them in this release. This is separate from the miscellaneous
   commits, which have no JIRA issue related to them.
 * All Git commits associated with a JIRA issue. This is particularly
   helpful in mapping a solution to a particular problem, especially
   when multiple commits were needed to resolve an issue.

REP-13 #close
Reported by: Matt Jordan

Change-Id: I97275322fe5b8067e2c7317b6fbbfab532e6cd48
---
D release-summary.py
A release_summary.py
2 files changed, 709 insertions(+), 621 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/repotools refs/changes/79/179/6
-- 
To view, visit https://gerrit.asterisk.org/179
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I97275322fe5b8067e2c7317b6fbbfab532e6cd48
Gerrit-PatchSet: 6
Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list