ENV var should take precedence over command argument

Author: r-sierra

README.md

@@ -291,7 +291,9 @@ The `tailwindcss:build` task is automatically attached to the `test:prepare` Rak
 If you want unminified assets, you can:
 
 - pass a `debug` argument to the rake task, i.e. `rails tailwindcss:build[debug]` or `rails tailwindcss:watch[debug]`.
-- set an environment variable named `TAILWINDCSS_DEBUG` with a value of `true` or `1` 
+- set an environment variable named `TAILWINDCSS_DEBUG` with a non-blank value
+
+If both values are set, the environment variable will take precedence over the rake task argument.  
 
 ### Live rebuild
 

lib/tailwindcss/commands.rb

@@ -4,7 +4,7 @@ module Tailwindcss
   module Commands
     class << self
       def compile_command(debug: false, **kwargs)
-        debug ||= ["true", "1"].include?(ENV["TAILWINDCSS_DEBUG"])
+        debug = ENV["TAILWINDCSS_DEBUG"].present? unless ENV["TAILWINDCSS_DEBUG"].nil?
         rails_root = defined?(Rails) ? Rails.root : Pathname.new(Dir.pwd)
 
         command = [

test/lib/tailwindcss/commands_test.rb

@@ -36,23 +36,21 @@ def setup
   test ".compile_command debug environment variable" do
     begin
       Rails.stub(:root, File) do # Rails.root won't work in this test suite
-        ENV["TAILWINDCSS_DEBUG"] = "0"
+        ENV["TAILWINDCSS_DEBUG"] = ""
         actual = Tailwindcss::Commands.compile_command
         assert_kind_of(Array, actual)
         assert_includes(actual, "--minify")
 
-        ENV["TAILWINDCSS_DEBUG"] = "1"
-        actual = Tailwindcss::Commands.compile_command
+        actual = Tailwindcss::Commands.compile_command(debug: true)
         assert_kind_of(Array, actual)
-        refute_includes(actual, "--minify")
+        assert_includes(actual, "--minify")
 
-        ENV["TAILWINDCSS_DEBUG"] = "false"
+        ENV["TAILWINDCSS_DEBUG"] = "any non-blank value"
         actual = Tailwindcss::Commands.compile_command
         assert_kind_of(Array, actual)
-        assert_includes(actual, "--minify")
+        refute_includes(actual, "--minify")
 
-        ENV["TAILWINDCSS_DEBUG"] = "true"
-        actual = Tailwindcss::Commands.compile_command
+        actual = Tailwindcss::Commands.compile_command(debug: true)
         assert_kind_of(Array, actual)
         refute_includes(actual, "--minify")
       end