It is permissible to have a layer's README reference another README. If
this is the case, avoid other checks. Do this by checking for the word
README (case insensitive) in the README file. This might be too
permissive, but follows the pattern of looking for exact words (like
"patch" or "maintainer") in READMEs.
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
data = f.read()
self.assertTrue(data,
msg="Layer contains a README file but it is empty.")
- self.assertIn('maintainer',data)
+
+ # If a layer's README references another README, then the checks below are not valid
+ if re.search('README', data, re.IGNORECASE):
+ return
+
+ self.assertIn('maintainer', data)
self.assertIn('patch',data)
# Check that there is an email address in the README
email_regex = re.compile(r"[^@]+@[^@]+")