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

Update Gemfile to use fastlane fork #1537

Merged
merged 2 commits into from
Aug 29, 2023
Merged

Update Gemfile to use fastlane fork #1537

merged 2 commits into from
Aug 29, 2023

Conversation

ayoy
Copy link
Collaborator

@ayoy ayoy commented Aug 23, 2023

Task/Issue URL: https://app.asana.com/0/1203301625297703/1203772337302927/f

Description:
Update to DDG's fork with a fix for fastlane/fastlane#21003

Steps to test this PR:
We'll really be able to test it while releasing, but for now you can e.g. try bundle exec fastlane release_testflight, having previously commented out the upload_to_testflight line so that it only builds the app.


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

@loremattei loremattei self-requested a review August 24, 2023 08:06
@loremattei loremattei self-assigned this Aug 24, 2023
Copy link
Contributor

@loremattei loremattei left a comment

Choose a reason for hiding this comment

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

Code changes look good.
I smoke tested as described and it worked.
We'll have the final test only during an actual release, but for now, I'd say ship it.

@ayoy ayoy assigned ayoy and unassigned loremattei Aug 29, 2023
@ayoy ayoy merged commit f0bf3a8 into develop Aug 29, 2023
@ayoy ayoy deleted the dominik/fastlane-fork branch August 29, 2023 06:49
samsymons added a commit that referenced this pull request Aug 30, 2023
# By Diego Rey Mendez (4) and others
# Via Diego Rey Mendez
* develop:
  Fixes the BSK version manually #3
  Fixes the BSK version manually #2
  Fixes the BSK version manually
  Fix auth token issues (#1563)
  Bye Catalina (#1501)
  exempt apps.facebook.com & standard.co.uk for CTL (#1505)
  Update Gemfile to use fastlane fork (#1537)
  PopUp window positioning (#1551)
  Fix AdClickAttribution Mock causing test failures (#1538)
  Bump version to 1.54.0 (55)
  Update embedded files
  Show privacy dashboard for duckduckgo serp (#1521)
  Use unencrypted password for deduplicated Credentials objects (#1554)
  DBP: Add missing selector properties to CCF (#1555)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants