post_version.py: invert `is_point` into `is_first_release` to make its purpose clearer
authorEric Engestrom <eric@engestrom.ch>
Wed, 6 May 2020 22:19:51 +0000 (00:19 +0200)
committerMarge Bot <eric+marge@anholt.net>
Wed, 13 May 2020 21:35:24 +0000 (21:35 +0000)
Signed-off-by: Eric Engestrom <eric@engestrom.ch>
Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4929>

bin/post_version.py

index 7ba24d8f7378b607df3b59768c9ecfa2ab16e6b9..446fae9880b200056ee0747dd404dffeb411cd94 100755 (executable)
@@ -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:
 
 
 def is_release_candidate(version: str) -> bool:
@@ -48,7 +48,7 @@ def branch_name(version: str) -> str:
     return f'{major}.{minor}'
 
 
     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)
     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}"
     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.")
         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
 
     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()
 
     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',
     update_release_notes(args.version)
     update_calendar(args.version)
     subprocess.run(['git', 'commit', '-m',