Skip to content

Commit

Permalink
Modify default bz_regex to support RH standards
Browse files Browse the repository at this point in the history 
Signed-off-by: Mattia Verga <mattia.verga@tiscali.it>
  • Loading branch information
@mattiaverga @mergify
mattiaverga authored and mergify[bot] committed Apr 22, 2020
1 parent 14b85da commit f2f8413
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion bodhi/server/config.py
View file
Expand Up @@ -279,7 +279,7 @@ class BodhiConfig(dict):
'value': ['Fedora', 'Fedora EPEL', 'Fedora Modules'],
'validator': _generate_list_validator(',')},
'bz_regex': {
'value': (r'(?:fix(?:es)?|close(?:s)?)\s'
'value': (r'(?:fix(?:es)?|close(?:s)?|resolve(?:s)?(?:\:)?)\s'
r'(?:fedora|epel|rh(?:bz)?)#(\d{5,})'),
'validator': str},
'bz_server': {
Expand Down

1 comment on commit f2f8413

@onosek
Copy link
Contributor

@onosek onosek commented on f2f8413 Jun 25, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mattiaverga
current expression

r'(?:fix(?:es)?|close(?:s)?|resolve(?:s)?(?:\:)?)\s(?:fedora|epel|rh(?:bz)?)#(\d{5,})':
resolves: rh#10000 OK
fixes: rh#10000    FAIL
closes: rh#10000   FAIL

I suggest minor update:
r'(?:fix(?:es)?|close(?:s)?|resolve(?:s)?)(?:\:)?\s(?:fedora|epel|rh(?:bz)?)#(\d{5,})

Please sign in to comment.