Patchwork D1097: releasenotes: don't abort is there is a bad formatted entry for releasenotes

login
register
mail settings
Submitter phabricator
Date Oct. 16, 2017, 6:29 p.m.
Message ID <3029c216936323bc8729a9de2b7d38d5@localhost.localdomain>
Download mbox | patch
Permalink /patch/25028/
State Not Applicable
Headers show

Comments

phabricator - Oct. 16, 2017, 6:29 p.m.
pulkit updated this revision to Diff 2833.
pulkit edited the summary of this revision.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D1097?vs=2787&id=2833

REVISION DETAIL
  https://phab.mercurial-scm.org/D1097

AFFECTED FILES
  hgext/releasenotes.py

CHANGE DETAILS




To: pulkit, #hg-reviewers, yuja
Cc: yuja, mercurial-devel

Patch

diff --git a/hgext/releasenotes.py b/hgext/releasenotes.py
--- a/hgext/releasenotes.py
+++ b/hgext/releasenotes.py
@@ -24,6 +24,7 @@ 
     config,
     error,
     minirst,
+    node,
     registrar,
     scmutil,
     util,
@@ -333,9 +334,8 @@ 
 
             # TODO consider using title as paragraph for more concise notes.
             if not paragraphs:
-                raise error.Abort(_('could not find content for release note '
-                                    '%s') % directive)
-
+                repo.ui.warn(_("error parsing releasenotes for revision: "
+                               "'%s'\n") % node.hex(ctx.node()))
             if title:
                 notes.addtitleditem(directive, title, paragraphs)
             else: