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

login
register
mail settings
Submitter phabricator
Date Oct. 17, 2017, 2:16 p.m.
Message ID <0e765d940c2595342e86037c3df11517@localhost.localdomain>
Download mbox | patch
Permalink /patch/25099/
State Not Applicable
Headers show

Comments

phabricator - Oct. 17, 2017, 2:16 p.m.
This revision was automatically updated to reflect the committed changes.
Closed by commit rHGa542ad320adb: releasenotes: don&#039;t abort is there is a bad formatted entry for releasenotes (authored by pulkit, committed by ).

REPOSITORY
  rHG Mercurial

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

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,
@@ -325,9 +326,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: