Merge pull request #314 from Shopify/ryanb-verify-failure-message

Improve dsl --verify failure message a bit
Shopify · May 18, 2021 · 8f04dd4 · 8f04dd4
2 parents 6503426 + 567a787
commit 8f04dd4
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/lib/tapioca/generator.rb b/lib/tapioca/generator.rb
@@ -601,8 +601,9 @@ def perform_dsl_verification(dir)
       if diff.empty?
         say("Nothing to do, all RBIs are up-to-date.")
       else
-        say("RBI files are out-of-date, please run:")
-        say("  `#{Config::DEFAULT_COMMAND} dsl`")
+        say("RBI files are out-of-date. In your development environment, please run:", :green)
+        say("  `#{Config::DEFAULT_COMMAND} dsl`", [:green, :bold])
+        say("Once it is complete, be sure to commit and push any changes", :green)
 
         say("")
 

diff --git a/spec/tapioca/cli_spec.rb b/spec/tapioca/cli_spec.rb
@@ -642,8 +642,9 @@ class Image
             Checking for out-of-date RBIs...
 
 
-            RBI files are out-of-date, please run:
+            RBI files are out-of-date. In your development environment, please run:
               `bin/tapioca dsl`
+            Once it is complete, be sure to commit and push any changes
 
             Reason:
               File(s) added:
@@ -692,8 +693,9 @@ class Image
             Checking for out-of-date RBIs...
 
 
-            RBI files are out-of-date, please run:
+            RBI files are out-of-date. In your development environment, please run:
               `bin/tapioca dsl`
+            Once it is complete, be sure to commit and push any changes
 
             Reason:
               File(s) changed: