Use atomic file operation to unlink file (#336)

Fix `Lint/NonAtomicFileOperation` unsafe offense `FileUtils.rm_f` is an alias of `FileUtils.safe_unlink`
ifad · Jan 30, 2025 · 583f31c · 583f31c
1 parent 9fb74a2
commit 583f31c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
diff --git a/lib/document.rb b/lib/document.rb
@@ -159,7 +159,7 @@ def set_current(version)
       raise InvalidVersion.new unless /^v\d+$/.match?(version)
 
       # need to do this, or ln_s will put the symlink *into* the old dir
-      File.unlink directory + CURRENT if File.exist? directory + CURRENT
+      FileUtils.rm_f directory + CURRENT
       FileUtils.ln_s version, directory + CURRENT, force: true
     end