scripts/send-qa-email: invert boolean logic for release check

is_non_release_version has an inverted logic which makes its reuse quite
confusing

Transform it as is_release_version and let caller do the negation if needed

Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Alexis Lothoré 2023-03-13 15:51:40 +01:00 committed by Richard Purdie
parent b913d47991
commit 0a815f4032

View File

@ -14,15 +14,15 @@ import re
import utils
def is_non_release_version(version):
def is_release_version(version):
p = re.compile('\d{8}-\d+')
return p.match(version) is not None
return p.match(version) is None
def get_previous_tag(targetrepodir, version):
previousversion = None
previousmilestone = None
if version:
if is_non_release_version(version):
if not is_release_version(version):
return subprocess.check_output(["git", "describe", "--abbrev=0"], cwd=targetrepodir).decode('utf-8').strip()
compareversion, comparemilestone, _ = utils.get_version_from_string(version)
compareversionminor = compareversion[-1]