jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/439525 ) Change subject: [bugfix] Set nochange to False if treat_result is 'done' ...................................................................... [bugfix] Set nochange to False if treat_result is 'done' Bug: T196675 Change-Id: I78fa4719c43b4ffdd6a06f0e9646a733e6449adc --- M scripts/solve_disambiguation.py 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified diff --git a/scripts/solve_disambiguation.py b/scripts/solve_disambiguation.py index 7c4819e..81b2f05 100755 --- a/scripts/solve_disambiguation.py +++ b/scripts/solve_disambiguation.py @@ -713,9 +713,7 @@ treat_result = self.treat_disamb_only(refPage, page) if treat_result == 'nextpage': return - elif treat_result in ('nochange', 'done'): - continue - elif nochange: + if treat_result == 'done': nochange = False if nochange: -- To view, visit https://gerrit.wikimedia.org/r/439525 To unsubscribe, or for help writing mail filters, visit https://gerrit.wikimedia.org/r/settings Gerrit-Project: pywikibot/core Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I78fa4719c43b4ffdd6a06f0e9646a733e6449adc Gerrit-Change-Number: 439525 Gerrit-PatchSet: 2 Gerrit-Owner: Xqt <info(a)gno.de> Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com> Gerrit-Reviewer: Dvorapa <dvorapa(a)seznam.cz> Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com> Gerrit-Reviewer: Matěj Suchánek <matejsuchanek97(a)gmail.com> Gerrit-Reviewer: Xqt <info(a)gno.de> Gerrit-Reviewer: Zoranzoki21 <zorandori4444(a)gmail.com> Gerrit-Reviewer: jenkins-bot