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

[Tooling] Update fastlane + remove workaround for match mutation #18417

Merged
merged 2 commits into from
Apr 22, 2022

Conversation

AliSoftware
Copy link
Contributor

@AliSoftware AliSoftware commented Apr 21, 2022

What

To Test

  • Run bundle exec fastlane update_certs_and_profiles and verify it doesn't crash

@AliSoftware AliSoftware added the Tooling Build, Release, and Validation Tools label Apr 21, 2022
@AliSoftware AliSoftware added this to the 19.8 milestone Apr 21, 2022
@AliSoftware AliSoftware requested a review from a team April 21, 2022 19:45
@AliSoftware AliSoftware self-assigned this Apr 21, 2022
Comment on lines -50 to -52
# - https://github.com/fastlane/fastlane/issues/11324
# - https://github.com/fastlane/fastlane/blob/b817148d4967ab6e38b8f1a8b81d1b3e60d6e645/match/lib/match/runner.rb#L84-L86
# - https://buildkite.com/automattic/wordpress-ios/builds/6240#ad789b9f-b744-4867-8d58-95921c58a3cf/2491-2756
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed by fastlane/fastlane#20125 since, which just landed in 2.205.2.

@wpmobilebot
Copy link
Contributor

You can test the changes in Jetpack from this Pull Request by:
  • Clicking here or scanning the QR code below
  • Then installing the build number pr18417-9501859 from App Center on your iPhone

The .ipa file can also be downloaded directly here.
If you need access to App Center, please ask a maintainer to add you.

@wpmobilebot
Copy link
Contributor

You can test the changes in WordPress from this Pull Request by:
  • Clicking here or scanning the QR code below
  • Then installing the build number pr18417-9501859 from App Center on your iPhone

The .ipa file can also be downloaded directly here.
If you need access to App Center, please ask a maintainer to add you.

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.

🎉

Yes! I've been checking for a Fastlane release daily waiting for your fix to ship. Glad it's finally here.

@AliSoftware AliSoftware merged commit a4e72d8 into trunk Apr 22, 2022
@AliSoftware AliSoftware deleted the tooling/update-fastlane-match-fix branch April 22, 2022 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tooling Build, Release, and Validation Tools
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants