jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/446227 )
Change subject: [bugfix] Recognize more than one escape sequences
......................................................................
[bugfix] Recognize more than one escape sequences
Change-Id: I0dfe6eac74bf66ba3827138c003ac38959b4b88d
---
M scripts/maintenance/diff_checker.py
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
Dalba: Looks good to me, approved
jenkins-bot: Verified
diff --git a/scripts/maintenance/diff_checker.py b/scripts/maintenance/diff_checker.py
index 081236e..59d12b8 100644
--- a/scripts/maintenance/diff_checker.py
+++ b/scripts/maintenance/diff_checker.py
@@ -78,8 +78,7 @@
elif quote == "'":
if (
'r' not in match.group('prefix')
- and r'\'' in string
- and not string.endswith(r'\'')
+ and string.count(r'\'') -
int(string.endswith(r'\''))
and '"' not in string
):
print_error(
--
To view, visit
https://gerrit.wikimedia.org/r/446227
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: I0dfe6eac74bf66ba3827138c003ac38959b4b88d
Gerrit-Change-Number: 446227
Gerrit-PatchSet: 1
Gerrit-Owner: Xqt <info(a)gno.de>
Gerrit-Reviewer: Dalba <dalba.wiki(a)gmail.com>
Gerrit-Reviewer: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Zoranzoki21 <zorandori4444(a)gmail.com>
Gerrit-Reviewer: jenkins-bot