Merge pull request basecamp#746 from igor-alexandrov/file-join

Replaced string interpolations with File.join to build paths

lib/kamal.rb

@@ -5,6 +5,6 @@ module Kamal
 require "zeitwerk"
 
 loader = Zeitwerk::Loader.for_gem
-loader.ignore("#{__dir__}/kamal/sshkit_with_ext.rb")
+loader.ignore(File.join(__dir__, "kamal", "sshkit_with_ext.rb"))
 loader.setup
 loader.eager_load # We need all commands loaded.

lib/kamal/commands/auditor.rb

@@ -21,7 +21,7 @@ def reveal
  def audit_log_file
  file = [ config.service, config.destination, "audit.log" ].compact.join("-")
 
- "#{config.run_directory}/#{file}"
+ File.join(config.run_directory, file)
  end
 
  def audit_tags(**details)

lib/kamal/commands/hook.rb

@@ -9,6 +9,6 @@ def hook_exists?(hook)
 
  private
  def hook_file(hook)
- "#{config.hooks_path}/#{hook}"
+ File.join(config.hooks_path, hook)
  end
 end

lib/kamal/configuration.rb

@@ -222,7 +222,7 @@ def asset_path
 
 
  def host_env_directory
- "#{run_directory}/env"
+ File.join(run_directory, "env")
  end
 
  def env

lib/kamal/configuration/accessory.rb

@@ -134,7 +134,7 @@ def remote_directories_as_volumes
  end
 
  def expand_host_path(host_path)
- absolute_path?(host_path) ? host_path : "#{service_data_directory}/#{host_path}"
+ absolute_path?(host_path) ? host_path : File.join(service_data_directory, host_path)
  end
 
  def absolute_path?(path)