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

Use S3 for fastlane match #12692

Merged
merged 2 commits into from
May 14, 2024
Merged

Conversation

crazytonyli
Copy link
Contributor

What says in the title.

You can verify this change by running bundle exec fastlane ios alpha_code_signing && bundle exec fastlane ios appstore_code_signing locally.

@crazytonyli crazytonyli requested a review from a team May 10, 2024 05:07
@wpmobilebot
Copy link
Collaborator

WooCommerce iOS📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS WooCommerce iOS
Build Numberpr12692-9100001
Version18.5
Bundle IDcom.automattic.alpha.woocommerce
Commit9100001
App Center BuildWooCommerce - Prototype Builds #9007
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@crazytonyli crazytonyli added the category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc. label May 10, 2024
@crazytonyli crazytonyli added this to the 18.6 milestone May 10, 2024
@joshheald joshheald modified the milestones: 18.6, 18.7 May 10, 2024
Copy link
Contributor

@mokagio mokagio left a comment

Choose a reason for hiding this comment

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

What do you think of taking the chance to:

Of course, those are "refactors" as they don't change the behavior. Approving to unblock.

secrets_directory = File.join(Dir.home, '.configure', 'woocommerce-ios', 'secrets')
google_cloud_keys_file(File.join(secrets_directory, 'google_cloud_keys.json'))
storage_mode('s3')
s3_bucket('a8c-fastlane-match')
Copy link
Contributor

Choose a reason for hiding this comment

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

We'll need to add s3_region to address a runtime warning. See Automattic/simplenote-macos#1150 (comment)

Suggested change
s3_bucket('a8c-fastlane-match')
s3_bucket('a8c-fastlane-match')
s3_region('us-east-2')

@crazytonyli
Copy link
Contributor Author

@mokagio Thanks for the review. I'll focus on the fastlane match repo migration for now 🙂

@crazytonyli crazytonyli merged commit 0d956e0 into trunk May 14, 2024
26 of 29 checks passed
@crazytonyli crazytonyli deleted the tonyli-migrate-fastlane-match-to-use-s3 branch May 14, 2024 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: tooling Anything that involves building & maintaining the project, including scripts, `Fastfile`, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants