Skip to content

Commit 128054a

Browse files
committed
Fix RuboCop offenses
- Remove redundant begin blocks in upload loops - Disable Metrics/ClassLength for extended classes
1 parent 5620982 commit 128054a

2 files changed

Lines changed: 24 additions & 28 deletions

File tree

lib/fastlane/plugin/amazon_appstore/actions/upload_to_amazon_appstore_action.rb

Lines changed: 23 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
module Fastlane
55
module Actions
6-
class UploadToAmazonAppstoreAction < Action
6+
class UploadToAmazonAppstoreAction < Action # rubocop:disable Metrics/ClassLength
77
def self.run(params) # rubocop:disable Metrics/PerceivedComplexity
88
Helper::AmazonAppstoreHelper.setup(
99
timeout: params[:timeout]
@@ -182,19 +182,17 @@ def self.upload_images(params, edit_id, token)
182182
end
183183

184184
images.each do |image_path|
185-
begin
186-
Helper::AmazonAppstoreHelper.upload_image(
187-
app_id: params[:package_name],
188-
edit_id: edit_id,
189-
language: language,
190-
image_type: image_type,
191-
image_path: image_path,
192-
token: token
193-
)
194-
UI.message("Uploaded #{image_type} for #{language}: #{File.basename(image_path)}")
195-
rescue StandardError => e
196-
UI.error("Failed to upload #{image_type} for #{language}: #{e.message}")
197-
end
185+
Helper::AmazonAppstoreHelper.upload_image(
186+
app_id: params[:package_name],
187+
edit_id: edit_id,
188+
language: language,
189+
image_type: image_type,
190+
image_path: image_path,
191+
token: token
192+
)
193+
UI.message("Uploaded #{image_type} for #{language}: #{File.basename(image_path)}")
194+
rescue StandardError => e
195+
UI.error("Failed to upload #{image_type} for #{language}: #{e.message}")
198196
end
199197
end
200198
end
@@ -227,19 +225,17 @@ def self.upload_screenshots(params, edit_id, token)
227225
end
228226

229227
screenshots.each do |screenshot_path|
230-
begin
231-
Helper::AmazonAppstoreHelper.upload_image(
232-
app_id: params[:package_name],
233-
edit_id: edit_id,
234-
language: language,
235-
image_type: image_type,
236-
image_path: screenshot_path,
237-
token: token
238-
)
239-
UI.message("Uploaded #{image_type} for #{language}: #{File.basename(screenshot_path)}")
240-
rescue StandardError => e
241-
UI.error("Failed to upload #{image_type} for #{language}: #{e.message}")
242-
end
228+
Helper::AmazonAppstoreHelper.upload_image(
229+
app_id: params[:package_name],
230+
edit_id: edit_id,
231+
language: language,
232+
image_type: image_type,
233+
image_path: screenshot_path,
234+
token: token
235+
)
236+
UI.message("Uploaded #{image_type} for #{language}: #{File.basename(screenshot_path)}")
237+
rescue StandardError => e
238+
UI.error("Failed to upload #{image_type} for #{language}: #{e.message}")
243239
end
244240
end
245241
end

lib/fastlane/plugin/amazon_appstore/helper/amazon_appstore_helper.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ module Fastlane
77
UI = FastlaneCore::UI unless Fastlane.const_defined?(:UI)
88

99
module Helper
10-
class AmazonAppstoreHelper
10+
class AmazonAppstoreHelper # rubocop:disable Metrics/ClassLength
1111
BASE_URL = 'https://developer.amazon.com'
1212
AUTH_URL = 'https://api.amazon.com/auth/o2/token'
1313

0 commit comments

Comments
 (0)