Merge pull request #172 from excid3/build-tasks-namespace

Namespace helper methods for javascript:build tasks

Author: rafaelfranca

lib/tasks/jsbundling/build.rake

@@ -1,15 +1,15 @@
 namespace :javascript do
   desc "Install JavaScript dependencies"
   task :install do
-    command = install_command
+    command = Jsbundling::Tasks.install_command
     unless system(command)
       raise "jsbundling-rails: Command install failed, ensure #{command.split.first} is installed"
     end
   end
 
   desc "Build your JavaScript bundle"
   build_task = task :build do
-    command = build_command
+    command = Jsbundling::Tasks.build_command
     unless system(command)
       raise "jsbundling-rails: Command build failed, ensure `#{command}` runs without errors"
     end
@@ -18,20 +18,26 @@ namespace :javascript do
   build_task.prereqs << :install unless ENV["SKIP_YARN_INSTALL"] || ENV["SKIP_BUN_INSTALL"]
 end
 
-def install_command
-  return "bun install" if File.exist?('bun.lockb') || (tool_exists?('bun') && !File.exist?('yarn.lock'))
-  return "yarn install" if File.exist?('yarn.lock') || tool_exists?('yarn')
-  raise "jsbundling-rails: No suitable tool found for installing JavaScript dependencies"
-end
+module Jsbundling
+  module Tasks
+    extend self
 
-def build_command
-  return "bun run build" if File.exist?('bun.lockb') || (tool_exists?('bun') && !File.exist?('yarn.lock'))
-  return "yarn build" if File.exist?('yarn.lock') || tool_exists?('yarn')
-  raise "jsbundling-rails: No suitable tool found for building JavaScript"
-end
+    def install_command
+      return "bun install" if File.exist?('bun.lockb') || (tool_exists?('bun') && !File.exist?('yarn.lock'))
+      return "yarn install" if File.exist?('yarn.lock') || tool_exists?('yarn')
+      raise "jsbundling-rails: No suitable tool found for installing JavaScript dependencies"
+    end
 
-def tool_exists?(tool)
-  system "command -v #{tool} > /dev/null"
+    def build_command
+      return "bun run build" if File.exist?('bun.lockb') || (tool_exists?('bun') && !File.exist?('yarn.lock'))
+      return "yarn build" if File.exist?('yarn.lock') || tool_exists?('yarn')
+      raise "jsbundling-rails: No suitable tool found for building JavaScript"
+    end
+
+    def tool_exists?(tool)
+      system "command -v #{tool} > /dev/null"
+    end
+  end
 end
 
 unless ENV["SKIP_JS_BUILD"]