post_version.py: invert `is_point` into `is_first_release` to make its purpose clearer

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>
This commit is contained in:
Eric Engestrom 2020-05-07 00:19:51 +02:00 committed by Marge Bot
parent 5fba85bcb8
commit ae26149e2e
1 changed files with 7 additions and 9 deletions

View File

@ -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',