Merge pull request #179540 from Homebrew/rubocop-fix-rmrf-usage

*casks: : Avoid using `FileUtils#rm_rf`, `FileUtils#rm_f` or `Pathname#rmtree`
Homebrew · Jul 14, 2024 · b3992ec · b3992ec
2 parents 2998a15 + d07ee1d
commit b3992ec
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/Casks/a/android-ndk.rb b/Casks/a/android-ndk.rb
@@ -35,7 +35,7 @@
  ].each { |link_name| binary shimscript, target: link_name }
 
  uninstall_postflight do
- FileUtils.rm_f("#{HOMEBREW_PREFIX}/share/android-ndk")
+ FileUtils.rm("#{HOMEBREW_PREFIX}/share/android-ndk")
  end
 
  # No zap stanza required

diff --git a/Casks/a/android-sdk.rb b/Casks/a/android-sdk.rb
@@ -30,7 +30,7 @@
  end
 
  uninstall_postflight do
- FileUtils.rm_f("#{HOMEBREW_PREFIX}/share/android-sdk")
+ FileUtils.rm("#{HOMEBREW_PREFIX}/share/android-sdk")
  end
 
  zap trash: "~/.android"

diff --git a/Casks/f/flutter.rb b/Casks/f/flutter.rb
@@ -26,7 +26,7 @@
  end
 
  uninstall_postflight do
- FileUtils.rm_f("#{HOMEBREW_PREFIX}/share/flutter")
+ FileUtils.rm("#{HOMEBREW_PREFIX}/share/flutter")
  end
 
  zap trash: "~/.flutter"

diff --git a/Casks/m/minecraft-server.rb b/Casks/m/minecraft-server.rb
@@ -42,7 +42,7 @@
  end
 
  uninstall_preflight do
- FileUtils.rm_f eula_file
+ FileUtils.rm(eula_file)
  end
 
  zap trash: config_dir

diff --git a/Casks/w/wezterm.rb b/Casks/w/wezterm.rb
@@ -40,7 +40,7 @@
  staged_subfolder = staged_path.glob(["WezTerm-*", "wezterm-*"]).first
  if staged_subfolder
  FileUtils.mv(staged_subfolder/"WezTerm.app", staged_path)
- FileUtils.rm_rf(staged_subfolder)
+ FileUtils.rm_r(staged_subfolder)
  end
  end
 

diff --git a/Casks/w/[email protected] b/Casks/w/[email protected]
@@ -26,7 +26,7 @@
  staged_subfolder = staged_path.glob(["WezTerm-*", "wezterm-*"]).first
  if staged_subfolder
  FileUtils.mv(staged_subfolder/"WezTerm.app", staged_path)
- FileUtils.rm_rf(staged_subfolder)
+ FileUtils.rm_r(staged_subfolder)
  end
  end