Fix lint issues

pace · Apr 5, 2023 · 2366e00 · 2366e00
1 parent a18a808
commit 2366e00
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 3 deletions.
diff --git a/deliver/lib/deliver/upload_app_clip_default_experience_header_images.rb b/deliver/lib/deliver/upload_app_clip_default_experience_header_images.rb
@@ -80,7 +80,6 @@ def upload(app_clip_default_experience, localizations, app_clip_header_images)
       Helper.show_loading_indicator("Waiting for all the app clip header images to finish being processed...")
       wait_for_complete(app_clip_default_experience.id)
       Helper.hide_loading_indicator
-      # TODO: implement function similar to AppScreenshot's `retry_upload_screenshots_if_needed` for robustness
 
       UI.message("Successfully uploaded all app clip header images")
     end

diff --git a/deliver/lib/deliver/upload_app_clip_default_experience_metadata.rb b/deliver/lib/deliver/upload_app_clip_default_experience_metadata.rb
@@ -29,7 +29,7 @@ def upload_metadata(options)
 
       UI.important("Will begin uploading app clip default experience metadata for '#{version.version_string}' on App Store Connect")
 
-      # TODO: support handling more than one app clip target per app
+      # Currently only one app clip target per app is supported
       app_clips = app.get_app_clips
       app_clip = app_clips.first
 

diff --git a/spaceship/lib/spaceship/connect_api/models/app_clip_header_image.rb b/spaceship/lib/spaceship/connect_api/models/app_clip_header_image.rb
@@ -99,7 +99,6 @@ def self.create(client: nil, app_clip_default_experience_localization_id: nil, p
             attributes: post_attributes
           ).first
         rescue => error
-          # TODO: handle this error like how `app_screenshot` model does it
           puts("ERROR: Unable to create app clip header image reservation, error: #{error}")
         end