From be2a3f520c0fbd98133fae7665e11460436cc21d Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Tue, 16 Jun 2020 16:37:35 -0500 Subject: [PATCH] Add an author exception for OpenStack Ansible SIG The current author line wants to match a github author id. But some people, including the OpenStack project, do not use github, and additionally do not claim individual ownership but instead group ownership. Since there are already a couple of hard-coded examples in the regex, just add one more. Alternately we could come up with some mechanism to indicate that the author is purposely not listing a github id, but that seems a bit heavywight. --- .../_data/sanity/validate-modules/validate_modules/schema.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/lib/ansible_test/_data/sanity/validate-modules/validate_modules/schema.py b/test/lib/ansible_test/_data/sanity/validate-modules/validate_modules/schema.py index d6b9c5c84fb..2fe1c8fdb3c 100644 --- a/test/lib/ansible_test/_data/sanity/validate-modules/validate_modules/schema.py +++ b/test/lib/ansible_test/_data/sanity/validate-modules/validate_modules/schema.py @@ -19,8 +19,9 @@ any_string_types = Any(*string_types) # author: First Last (@name) [optional anything] # "Ansible Core Team" - Used by the Bot # "Michael DeHaan" - nop +# "OpenStack Ansible SIG" - OpenStack does not use GitHub # "Name (!UNKNOWN)" - For the few untraceable authors -author_line = re.compile(r'^\w.*(\(@([\w-]+)\)|!UNKNOWN)(?![\w.])|^Ansible Core Team$|^Michael DeHaan$') +author_line = re.compile(r'^\w.*(\(@([\w-]+)\)|!UNKNOWN)(?![\w.])|^Ansible Core Team$|^Michael DeHaan$|^OpenStack Ansible SIG$') def sequence_of_sequences(min=None, max=None):