NextERP/runbot
5
0
Fork 0
mirror of https://github.com/odoo/runbot.git synced 2025-03-15 23:45:44 +07:00
Code Issues Actions Packages Projects Releases Wiki Activity

[IMP] runbot_merge: make space characters in regexes clearer

Browse Source 
pycharm is a bit dumb so it considers `[ ]` to be unnecessary even in
`re.VERBOSE` regexes. But it has a bit of a point in that it's not
super clear, so inject the space characters by (unicode) name, the
behaviour should be the exact same but the regexes should be slightly
less opaque.
This commit is contained in:
Xavier Morel 2025-01-23 15:52:03 +01:00
parent 60b84490c6
commit edaf507863
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
runbot_merge/models/stagings_create.py
View File

@ -691,7 +691,7 @@ def add_self_references(
))
BREAK = re.compile(r'''
[ ]{0,3} # 0-3 spaces of indentation
\N{SPACE}{0,3} # 0-3 spaces of indentation
# followed by a sequence of three or more matching -, _, or * characters,
# each followed optionally by any number of spaces or tabs
# so needs to start with a _, - or *, then have at least 2 more such
@ -701,9 +701,9 @@ BREAK = re.compile(r'''
[ \t]*
''', flags=re.VERBOSE)
SETEX_UNDERLINE = re.compile(r'''
[ ]{0,3} # no more than 3 spaces indentation
\N{SPACE}{0,3} # no more than 3 spaces indentation
[-=]+ # a sequence of = characters or a sequence of - characters
[ ]* # any number of trailing spaces
\N{SPACE}* # any number of trailing spaces
# we don't care about "a line containing a single -" because we want to
# disambiguate SETEX headings from thematic breaks, and thematic breaks have
# 3+ -. Doesn't look like GH interprets `- - -` as a line so yay...