Apply suggestions from code review

Co-authored-by: Gio Lodi <giovanni.lodi42@gmail.com>

Author: AliSoftware

lib/fastlane/plugin/wpmreleasetoolkit/actions/ios/ios_send_app_size_metrics.rb

@@ -97,35 +97,35 @@ def self.available_options
           FastlaneCore::ConfigItem.new(
             key: :app_name,
             env_name: 'FL_IOS_SEND_APP_SIZE_METRICS_APP_NAME',
-            description: 'The name of the app for which we are publishing metrics, to help filter by app in the dashboard',
+            description: 'The name of the app for which we are publishing metrics, to help with filtering and grouping',
             type: String,
             optional: false
           ),
           FastlaneCore::ConfigItem.new(
             key: :app_version,
             env_name: 'FL_IOS_SEND_APP_SIZE_METRICS_APP_VERSION',
-            description: 'The version of the app for which we are publishing metrics, to help filter by version in the dashboard',
+            description: 'The version of the app for which we are publishing metrics, to help with filtering and grouping',
             type: String,
             optional: false
           ),
           FastlaneCore::ConfigItem.new(
             key: :build_type,
             env_name: 'FL_IOS_SEND_APP_SIZE_METRICS_BUILD_TYPE',
-            description: 'The build configuration for which we are publishing metrics, to help filter by build config in the dashboard. E.g. `Debug`, `Release`',
+            description: 'The build configuration for which we are publishing metrics, to help with filtering and grouping. E.g. `Debug`, `Release`',
             type: String,
             optional: true
           ),
           FastlaneCore::ConfigItem.new(
             key: :source,
             env_name: 'FL_IOS_SEND_APP_SIZE_METRICS_SOURCE',
-            description: 'The type of event at the origin of that build, to help filter data in the dashboard. E.g. `pr`, `beta`, `final-release`',
+            description: 'The type of event at the origin of that build, to help with filtering and grouping. E.g. `pr`, `beta`, `final-release`',
             type: String,
             optional: true
           ),
           FastlaneCore::ConfigItem.new(
             key: :ipa_path,
             env_name: 'FL_IOS_SEND_APP_SIZE_METRICS_IPA_PATH',
-            description: 'The path to the .ipa to extract size information from',
+            description: 'The path to the `.ipa` to extract size information from',
             type: String,
             optional: false,
             default_value: Actions.lane_context[SharedValues::IPA_OUTPUT_PATH],
@@ -158,7 +158,7 @@ def self.authors
       end
 
       def self.is_supported?(platform)
-        [:ios, :mac].include? platform
+        platform == :ios
       end
     end
   end

lib/fastlane/plugin/wpmreleasetoolkit/helper/app_size_metrics_helper.rb

@@ -59,7 +59,7 @@ def send_metrics(to:, api_token:, use_gzip: true)
 
         # Allow using a `file:` URI for debugging
         if uri.is_a?(URI::File)
-          UI.message("Writing metrics payload to file #{uri.path} (instead of sending it to a server)")
+          UI.message("Writing metrics payload to file #{uri.path} (instead of sending it to a remote API endpoint)")
           File.write(uri.path, json_payload)
           return 201 # To make it easy at call site to check for pseudo-status code 200 even in non-HTTP cases
         end