Fix inspect should not write history entry

Signed-off-by: Paolo Di Tommaso <[email protected]>
nextflow-io · May 14, 2024 · c713ad5 · c713ad5
1 parent a5ecf8a
commit c713ad5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/CmdInspect.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/CmdInspect.groovy
@@ -85,6 +85,7 @@ class CmdInspect extends CmdBase {
         target.preview = true
         target.previewAction = this.&applyInspect
         target.ansiLog = false
+        target.skipHistoryFile = true
         // run it
         target.run()
     }

diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/CmdRun.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/CmdRun.groovy
@@ -266,6 +266,8 @@ class CmdRun extends CmdBase implements HubOptions {
     @Parameter(names=['-disable-jobs-cancellation'], description = 'Prevent the cancellation of child jobs on execution termination')
     Boolean disableJobsCancellation
 
+    Boolean skipHistoryFile
+
     Boolean getDisableJobsCancellation() {
         return disableJobsCancellation!=null
                 ?  disableJobsCancellation
@@ -362,7 +364,9 @@ class CmdRun extends CmdBase implements HubOptions {
         log.debug( '\n'+info )
 
         // -- add this run to the local history
-        runner.verifyAndTrackHistory(launcher.cliString, runName)
+        if( !skipHistoryFile )  {
+            runner.verifyAndTrackHistory(launcher.cliString, runName)
+        }
 
         // -- run it!
         runner.execute(scriptArgs, this.entryName)