0 7 3 catch exceptions (#151)

Don't let NextFlow die silently.
quiltdata · Oct 16, 2023 · b9d5c04 · b9d5c04
1 parent 888f4f4
commit b9d5c04
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## [0.7.3] 2023-10-16a
+
+- Catch and log exception on push failure
+
 ## [0.7.2] 2023-10-16
 
 - Throw exceptions on push failure

diff --git a/README.md b/README.md
@@ -75,8 +75,8 @@ From the command-line, do, e.g.:
 
 ```bash
 # export NXF_VER=23.04.3
-export NXF_PLUGINS_TEST_REPOSITORY=https://github.com/quiltdata/nf-quilt/releases/download/0.7.2/nf-quilt-0.7.2-meta.json
-nextflow run main.nf -plugins [email protected].2
+export NXF_PLUGINS_TEST_REPOSITORY=https://github.com/quiltdata/nf-quilt/releases/download/0.7.3/nf-quilt-0.7.3-meta.json
+nextflow run main.nf -plugins [email protected].3
 ```
 
 For Tower, you can use the "Pre-run script" to set the environment variables.

diff --git a/plugins/nf-quilt/src/main/nextflow/quilt/QuiltProduct.groovy b/plugins/nf-quilt/src/main/nextflow/quilt/QuiltProduct.groovy
@@ -138,7 +138,12 @@ ${nextflow}
         log.debug("setupReadme: $text")
         List<Map> quilt_summarize = setupSummarize()
         log.debug("setupSummarize: $quilt_summarize")
-        pkg.push(msg, meta)
+        try {
+            pkg.push(msg, meta)
+        }
+        catch (Exception e) {
+            log.error("publish failed: ${e.getMessage()}\n${e}", pkg.meta)
+        }
         print("SUCCESS: $pkg\n")
     }
 

diff --git a/plugins/nf-quilt/src/resources/META-INF/MANIFEST.MF b/plugins/nf-quilt/src/resources/META-INF/MANIFEST.MF
@@ -1,7 +1,7 @@
 Manifest-Version: 1.0
 Plugin-Class: nextflow.quilt.QuiltPlugin
 Plugin-Id: nf-quilt
-Plugin-Version: 0.7.2
+Plugin-Version: 0.7.3
 Plugin-Provider: Quilt Data
 Plugin-Requires: >=22.10.6