Merge "Improve gerrit checks messages"
This commit is contained in:
commit
5dd4dd54ab
@ -14,7 +14,6 @@ function check_team_acl {
|
|||||||
local failure=0
|
local failure=0
|
||||||
|
|
||||||
for config in $configs_list; do
|
for config in $configs_list; do
|
||||||
echo "Checking $config file..."
|
|
||||||
|
|
||||||
$OLDPWD/tools/normalize_acl.py $config all > $TMPDIR/normalized
|
$OLDPWD/tools/normalize_acl.py $config all > $TMPDIR/normalized
|
||||||
if ! diff -u $config $TMPDIR/normalized >>config_failures;
|
if ! diff -u $config $TMPDIR/normalized >>config_failures;
|
||||||
|
@ -129,9 +129,9 @@ def main():
|
|||||||
# sort of job-description (e.g. "foo-devstack-bar") or
|
# sort of job-description (e.g. "foo-devstack-bar") or
|
||||||
# a url ("foo.openstack.org")
|
# a url ("foo.openstack.org")
|
||||||
if re.search(r'(?<![-.])\b%s\b' % word, description):
|
if re.search(r'(?<![-.])\b%s\b' % word, description):
|
||||||
print("ERROR: description '%s': contains wrong word, "
|
print("ERROR: project %s, description '%s': "
|
||||||
"it should be '%s'" %
|
"contains wrong word '%s', it should be '%s'" %
|
||||||
(description, should_be))
|
(name, description, word, should_be))
|
||||||
found_errors += 1
|
found_errors += 1
|
||||||
|
|
||||||
if not description and repo_group in DESCRIPTION_REQUIRED:
|
if not description and repo_group in DESCRIPTION_REQUIRED:
|
||||||
|
Loading…
Reference in New Issue
Block a user