Merge pull request #61 from hathitrust/history_already_exists_error

Fix spurious error exit from `add_monthly_and_dump_redirects.rb`.
hathitrust · Oct 10, 2024 · af5e4ff · af5e4ff
2 parents 5cfa455 + c563a19
commit af5e4ff
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/bin/add_monthly_and_dump_redirects.rb b/bin/add_monthly_and_dump_redirects.rb
@@ -69,9 +69,10 @@ def usage
   exit 1
 end
 
+# Argo daily workflow calls this each day, it is not an error to have already done the work.
 if File.exist?(new_history_file)
-  LOGGER.error "#{new_history_file} already exists. Rename/delete it first"
-  exit 1
+  LOGGER.info "#{new_history_file} already exists. Exiting."
+  exit 0
 end
 
 LOGGER.info "Will read from #{old_history_file}"