From: Kevin Sandi <ksandi@contractor.linuxfoundation.org>
Date: Wed, 19 Jul 2023 23:28:41 +0000 (-0600)
Subject: Chore: add new lines required by pre-commit
X-Git-Tag: v0.37.6^0
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=85fdad945f0afaf8b1d94f8bdfaa15254f8ef695;p=releng%2Flftools.git

Chore: add new lines required by pre-commit

Signed-off-by: Kevin Sandi <ksandi@contractor.linuxfoundation.org>
Change-Id: I659776622fefceb233f3a444ec38f5174a5d5054
---

diff --git a/lftools/git/gerrit.py b/lftools/git/gerrit.py
index ebf890f6..0bc8a314 100644
--- a/lftools/git/gerrit.py
+++ b/lftools/git/gerrit.py
@@ -160,7 +160,9 @@ class Gerrit:
             else:
                 buildnode = "centos7-builder-2c-1g"
 
-        jinja_env = Environment(loader=PackageLoader("lftools.git"), autoescape=select_autoescape())
+        jinja_env = Environment(
+            loader=PackageLoader("lftools.git"), autoescape=select_autoescape(), keep_trailing_newline=True
+        )
         template = jinja_env.get_template("project.yaml")
         content = template.render(
             project_name_dashed=gerrit_project_dashed,
@@ -186,7 +188,9 @@ class Gerrit:
         """
         filename = ".gitreview"
 
-        jinja_env = Environment(loader=PackageLoader("lftools.git"), autoescape=select_autoescape())
+        jinja_env = Environment(
+            loader=PackageLoader("lftools.git"), autoescape=select_autoescape(), keep_trailing_newline=True
+        )
         template = jinja_env.get_template("gitreview")
         content = template.render(fqdn=fqdn, project_name=gerrit_project, default_branch=self.default_branch)
         log.debug(".gitreview contents:\n{}".format(content))
@@ -239,7 +243,9 @@ class Gerrit:
             except AttributeError:
                 log.error("Invalid nexus3_ports designated.")
 
-        jinja_env = Environment(loader=PackageLoader("lftools.git"), autoescape=select_autoescape())
+        jinja_env = Environment(
+            loader=PackageLoader("lftools.git"), autoescape=select_autoescape(), keep_trailing_newline=True
+        )
         template = jinja_env.get_template(params_path)
         config_params_content = template.render(project_dashed=project_dashed)
         log.debug("config-params.yaml contents:\n{}".format(config_params_content))
diff --git a/releasenotes/notes/pre-commit-fix-newlines-4c37e32a618b2483.yaml b/releasenotes/notes/pre-commit-fix-newlines-4c37e32a618b2483.yaml
new file mode 100644
index 00000000..507ea519
--- /dev/null
+++ b/releasenotes/notes/pre-commit-fix-newlines-4c37e32a618b2483.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+  - |
+    Adds new lines at EOF of templates used by lftools to generate automated
+    Gerrit changes after a new INFO.yaml is merged.