bin/post_version.py: Make the git commit as well.

Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
Reviewed-by: Juan A. Suarez <jasuarez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/2505>

diff --git a/bin/post_version.py b/bin/post_version.py
index 3fbb5e2107cc9c162e5345f98d4bc3b6b3394213..5cf5f48615e39916f6ccb3ae19d7ba26effc0a7f 100755 (executable)
--- a/bin/post_version.py
+++ b/bin/post_version.py
@@ -25,6 +25,8 @@ import argparse
 import calendar
 import datetime
 import pathlib
+import subprocess
+
 from lxml import (
     etree,
     html,
@@ -84,6 +86,7 @@ def update_index(is_point: bool, version: str, previous_version: str) -> None:
     root.insert(index, header)
 
     tree.write(p.as_posix(), method='html', pretty_print=True)
+    subprocess.run(['git', 'add', p])
 
 
 def update_release_notes(previous_version: str) -> None:
@@ -99,6 +102,7 @@ def update_release_notes(previous_version: str) -> None:
     ul.insert(0, li)
 
     tree.write(p.as_posix(), method='html', pretty_print=True)
+    subprocess.run(['git', 'add', p])
 
 
 def update_calendar(previous_version: str) -> None:
@@ -132,6 +136,7 @@ def update_calendar(previous_version: str) -> None:
         new.insert(0, td)
 
     tree.write(p.as_posix(), method='html', pretty_print=True)
+    subprocess.run(['git', 'add', p])
 
 
 def main() -> None:
@@ -145,6 +150,9 @@ def main() -> None:
     update_index(is_point, args.version, previous_version)
     update_release_notes(previous_version)
     update_calendar(previous_version)
+    subprocess.run(['git', 'commit', '-m',
+                    'docs: update calendar, add news item, and link releases '
+                    f'notes for {previous_version}'])
 
 
 if __name__ == "__main__":

diff --git a/bin/post_version_test.py b/bin/post_version_test.py
index 11b28061e576dde1d358d6807b9a5365c8a2be27..d0794a609f40fa5b21a39a7504b8f9fc24c580f5 100644 (file)
--- a/bin/post_version_test.py
+++ b/bin/post_version_test.py
@@ -27,6 +27,8 @@ import pytest
 from . import post_version
 
 
+# Mock out subprocess.run to avoid having git commits
+@mock.patch('bin.post_version.subprocess.run', mock.Mock())
 class TestUpdateCalendar:
 
     HEAD = textwrap.dedent("""\