Print changelogs and fail if git tag does not exist

Author: davegaeddert

CHANGELOG.md

@@ -9,6 +9,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Nothing right now.
 
+## [1.2.0] - 2020-09-06
+
+Allows changelog content for a version to be empty, while still creating a GitHub Release for the tag.
+
 ## [1.1.1] - 2020-09-06
 
 Added a `--no-tag-prefix` option to fix an empty issue with `--tag-prefix`.
@@ -22,6 +26,7 @@ Adds a `tag_prefix` option so that tags don't have to start with a "v". You can
 The first release! Includes the `sync` command which will sync your `CHANGELOG.md` to GitHub Release notes.
 
 [Unreleased]: https://github.com/dropseed/changerelease/compare/v1.1.1...HEAD
-[1.1.0]: https://github.com/dropseed/changerelease/compare/v1.1.0...v1.1.1
+[1.2.0]: https://github.com/dropseed/changerelease/compare/v1.1.1...v1.2.0
+[1.1.1]: https://github.com/dropseed/changerelease/compare/v1.1.0...v1.1.1
 [1.1.0]: https://github.com/dropseed/changerelease/compare/v1.0.0...v1.1.0
 [1.0.0]: https://github.com/dropseed/changerelease/releases/tag/v1.0.0

changerelease/cli.py

@@ -44,17 +44,32 @@ def sync(changelog, tag_prefix, no_tag_prefix, repo, api_url, token):
 
     cl = Changelog(changelog)
 
+    outline = "-" * 80
+
+    results = []
+
     for version in cl.versions:
         click.secho(f"\nSyncing version {version}", bold=True)
         version_contents = cl.parse_version_content(version)
-        if not version_contents:
-            click.secho(f"No content found for version {version}", fg="yellow")
+
+        click.echo(f"{outline}\n{version_contents or '(empty)'}\n{outline}")
 
         release = Release(repo, tag_prefix, version, requests_session)
-        release.sync(version_contents)
+        message, synced = release.sync(version_contents)
+
+        click.secho(message, fg="green" if synced else "red")
+
+        results.append(synced)
 
-    # TODO fail if tag doesn't exist yet? (continue w/ others though)
-    click.secho(f"\nGitHub releases on {repo} synced with {cl.path}", fg="green")
+    if all(results):
+        click.secho(
+            f"\nSynced {len(results)} GitHub Releases on {repo} with {cl.path}",
+            fg="green",
+        )
+    else:
+        failed = [x for x in results if not x]
+        click.secho(f"Failed to sync {len(failed)} versions", fg="red")
+        exit(1)
 
 
 if __name__ == "__main__":

changerelease/release.py

@@ -1,6 +1,3 @@
-import click
-
-
 class Release:
     def __init__(self, repo, tag_prefix, version, requests_session):
         self.repo = repo
@@ -57,24 +54,21 @@ def update_body(self, contents):
         return response.json()
 
     def sync(self, contents):
+        message = ""
+        synced = True
+
         if not self.exists():
             if self.tag_exists():
-                click.secho(
-                    f"Release for {self.version} does not exist. Creating it...",
-                    fg="green",
-                )
+                message = f"Release for {self.version} does not exist. Creating it..."
                 release_data = self.create(contents)
-                click.echo(release_data["html_url"])
+                message += "\n" + release_data["html_url"]
             else:
-                click.secho(
-                    f'Git tag "{self.version_tag}" not found. A tag needs to be pushed before we can create a release for it.',
-                    fg="red",
-                )
+                message = f'Git tag "{self.version_tag}" not found. A tag needs to be pushed before we can create a release for it.'
+                synced = False
         elif not self.body_matches(contents):
-            click.secho(
-                f"Release for {self.version} exists but the changelog doesn't match. Updating the release...",
-                fg="green",
-            )
+            message = f"Release for {self.version} exists but the changelog doesn't match. Updating the release..."
             self.update_body(contents)
         else:
-            click.echo("No change")
+            message = "Release is up-to-date"
+
+        return (message, synced)