Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[deliver] Initialize UploadMetadata with options #21861

Conversation

mollyIV
Copy link
Member

@mollyIV mollyIV commented Feb 9, 2024

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary.
  • I've added or updated relevant unit tests.

Motivation and Context

This pull request addresses the requested changes in here: #21742 (comment)

Please note this pull request is going to be merged into parent branch, not a main branch.

Description

To avoid redundant parameters setting and passing around, the UploadMetadata is not initialized with options.

Testing Steps

CI Pipelines ✔️

Copy link
Collaborator

@lacostej lacostej left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice cleanup. Let's merge this into @rogerluan 's PR

@lacostej lacostej merged commit 064f5c1 into rogerluan/add-exponential-backoff-to-retry Feb 9, 2024
3 checks passed
@lacostej lacostej deleted the upload-metadata-options-init branch February 9, 2024 17:24
lacostej pushed a commit that referenced this pull request Feb 9, 2024
… even when Apple servers are degraded (#21742)

* Add exponential backoff to retry mechanism that checks for app version existence.

* Add option for users to increase the number of max retries if needed.

* Fix linter issue complaining about long line.

* Set max wait time to 5 minutes.

* Adjust default number of retries to a more sensible number.

* [deliver] Initialize UploadMetadata with options (#21861)

* [deliver] Initialize UploadMetadata with options

* Fix linting

---------

Co-authored-by: Daniel Jankowski <daniell.jankowskii@gmail.com>
Copy link
Member

@rogerluan rogerluan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great thanks!! Thanks for this! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants