From: Eric Engestrom Date: Wed, 6 May 2020 22:19:51 +0000 (+0200) Subject: post_version.py: invert `is_point` into `is_first_release` to make its purpose clearer X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=ae26149e2e9ccac54d64ded9a9c0a8ae84cfd300;p=mesa.git post_version.py: invert `is_point` into `is_first_release` to make its purpose clearer Signed-off-by: Eric Engestrom Reviewed-by: Dylan Baker Part-of: --- diff --git a/bin/post_version.py b/bin/post_version.py index 7ba24d8f737..446fae9880b 100755 --- a/bin/post_version.py +++ b/bin/post_version.py @@ -33,8 +33,8 @@ from lxml import ( ) -def is_point_release(version: str) -> bool: - return not version.endswith('.0') +def is_first_release(version: str) -> bool: + return version.endswith('.0') def is_release_candidate(version: str) -> bool: @@ -48,7 +48,7 @@ def branch_name(version: str) -> str: return f'{major}.{minor}' -def update_index(is_point: bool, version: str) -> None: +def update_index(version: str) -> None: p = pathlib.Path(__file__).parent.parent / 'docs' / 'index.html' with p.open('rt') as f: tree = html.parse(f) @@ -64,11 +64,11 @@ def update_index(is_point: bool, version: str) -> None: a = etree.SubElement( body, 'a', attrib={'href': f'relnotes/{version}.html'}) a.text = f"Mesa {version}" - if is_point: - a.tail = " is released. This is a bug fix release." - else: + if is_first_release(version): a.tail = (" is released. This is a new development release. " "See the release notes for more information about this release.") + else: + a.tail = " is released. This is a bug fix release." root = news.getparent() index = root.index(news) + 1 @@ -134,9 +134,7 @@ def main() -> None: parser.add_argument('version', help="The released version.") args = parser.parse_args() - is_point = is_point_release(args.version) - - update_index(is_point, args.version) + update_index(args.version) update_release_notes(args.version) update_calendar(args.version) subprocess.run(['git', 'commit', '-m',