Merge pull request #44 from jcristau/diff

jcristau · web-flow · commit 30ce1f986a14 · 2022-07-04T15:27:35.000+02:00
importer: skip creating a commit/PR if no changes were made
diff --git a/src/mozxchannel/android/importer.py b/src/mozxchannel/android/importer.py
@@ -23,6 +23,12 @@ def main():
     subprocess.run(
         ["git", "-C", args.dest, "add", "-v", "-A"], check=True
     )
+    diff = subprocess.run(
+        ["git", "-C", args.dest, "diff", "--cached", "--exit-code"]
+    )
+    if diff.returncode == 0:
+        print("No changes")
+        return
     subprocess.run(
         ["git", "-C", args.dest, "commit", "-m", "Import l10n."], check=True
     )

Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,12 @@ def main():`
`23`	`23`	`subprocess.run(`
`24`	`24`	`["git", "-C", args.dest, "add", "-v", "-A"], check=True`
`25`	`25`	`)`
	`26`	`+ diff = subprocess.run(`
	`27`	`+ ["git", "-C", args.dest, "diff", "--cached", "--exit-code"]`
	`28`	`+ )`
	`29`	`+ if diff.returncode == 0:`
	`30`	`+ print("No changes")`
	`31`	`+ return`
`26`	`32`	`subprocess.run(`
`27`	`33`	`["git", "-C", args.dest, "commit", "-m", "Import l10n."], check=True`
`28`	`34`	`)`