Merge branch 'support-delete-given-release-type'

* support-delete-given-release-type:
  gh_release: Support deleting releases based on release type

Author: jcfr

CHANGES.md

@@ -11,6 +11,17 @@ CLI
 
   * ``release-notes``: Report a meaningful error if user omitted to set ``EDITOR`` env. variable.
 
+
+Features (CLI and Python API)
+-----------------------------
+
+* ``delete`` command:
+
+  * `delete`:
+
+    * Add support for ``--release-type``. Fix issue [#50](https://github.com/j0057/github-release/pull/50). Contributed by [@Flamefire](https://github.com/Flamefire).
+
+
 Issues (CLI and Python API)
 ---------------------------
 

README.md

@@ -185,6 +185,7 @@ It understands the following commands:
   --publish
   --prerelease
   --target-commitish TARGET_COMMITISH
+  --help
   [ASSET_PATTERN]...
 ```
 
@@ -199,8 +200,19 @@ It understands the following commands:
   --prerelease/--release
   --dry-run
   --verbose
+  --help
 ```
 
+* delete:
+
+```bash
+  --keep-pattern KEEP_PATTERN
+  --type [all, draft, prerelease, release]
+  --dry-run
+  --verbose
+  --help
+  [ASSET_PATTERN]...
+```
 
 ## ``asset`` command
 
@@ -464,4 +476,4 @@ It is covered by the Apache License, Version 2.0:
 http://www.apache.org/licenses/LICENSE-2.0
 
 The license file was added at revision 0393859 on 2017-02-12, but you may
-consider that the license applies to all prior revisions as well.
+consider that the license applies to all prior revisions as well.

github_release.py

@@ -255,6 +255,18 @@ def print_release_info(release, title=None, indent=""):
         print_asset_info(i, asset, indent=indent)
 
 
+def get_release_type(release):
+    """Return the type of the release
+
+    Either 'draft', 'prerelease' (no draft) or 'release' (neither)
+    """
+    if release['draft']:
+        return 'draft'
+    if release['prerelease']:
+        return 'prerelease'
+    return 'release'
+
+
 def get_releases(repo_name, verbose=False):
 
     releases = []
@@ -515,6 +527,7 @@ def gh_release_edit(repo_name, current_tag_name,
 @gh_release.command("delete")
 @click.argument("pattern")
 @click.option("--keep-pattern")
+@click.option("--release-type", type=click.Choice(['all', 'draft', 'prerelease', 'release']), default='all')
 @click.option("--dry-run", is_flag=True, default=False)
 @click.option("--verbose", is_flag=True, default=False)
 @click.pass_obj
@@ -524,7 +537,7 @@ def _cli_release_delete(*args, **kwargs):
 
 
 @_check_for_credentials
-def gh_release_delete(repo_name, pattern, keep_pattern=None,
+def gh_release_delete(repo_name, pattern, keep_pattern=None, release_type='all',
                       dry_run=False, verbose=False):
     releases = get_releases(repo_name)
     candidates = []
@@ -538,9 +551,13 @@ def gh_release_delete(repo_name, pattern, keep_pattern=None,
         if keep_pattern is not None:
             if fnmatch.fnmatch(release['tag_name'], keep_pattern):
                 continue
-        candidates.append(release['tag_name'])
-    for tag_name in candidates:
-        release = get_release(repo_name, tag_name)
+        if release_type != 'all' and release_type != get_release_type(release):
+            if verbose:
+                print('skipping release {0}: type {1} is not {2}'.format(
+                    release['tag_name'], get_release_type(release), release_type))
+            continue
+        candidates.append(release)
+    for release in candidates:
         print('deleting release {0}'.format(release['tag_name']))
         if dry_run:
             continue