Add guards against missing env vars

wordpress-mobile · May 14, 2021 · abfd65d · abfd65d
1 parent 2adf8c6
commit abfd65d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_get_app_version.rb b/lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_get_app_version.rb
@@ -3,6 +3,9 @@ module Actions
     class IosGetAppVersionAction < Action
       def self.run(params)
         require_relative '../../helper/ios/ios_version_helper.rb'
+
+        UI.user_error!('You need to set at least the PUBLIC_CONFIG_FILE env var to the path to the public xcconfig file') unless ENV['PUBLIC_CONFIG_FILE']
+
         Fastlane::Helper::Ios::VersionHelper.get_public_version
       end
 

diff --git a/lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_get_build_version.rb b/lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_get_build_version.rb
@@ -4,7 +4,10 @@ class IosGetBuildVersionAction < Action
       def self.run(params)
         require_relative '../../helper/ios/ios_version_helper.rb'
 
+        UI.user_error!('You need to set at least the PUBLIC_CONFIG_FILE env var to the path to the public xcconfig file') unless ENV['PUBLIC_CONFIG_FILE']
+
         if params[:internal]
+          UI.user_error!('You need to set the INTERNAL_CONFIG_FILE env var to the path to the internal xcconfig file') unless ENV['INTERNAL_CONFIG_FILE']
           Fastlane::Helper::Ios::VersionHelper.get_internal_version
         else
           Fastlane::Helper::Ios::VersionHelper.get_build_version