Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

Conversation

@diegoreymendez
Copy link
Contributor

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

Description:

Adds a file that's generated by our unit tests that's causing fastlane to fail.

Steps to test this PR:

  1. Just run the unit tests for our Developer ID and App Store tests.

Ensure no new file is created, and there are no modifications to existing files.


Internal references:

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

@diegoreymendez diegoreymendez marked this pull request as ready for review August 21, 2023 10:39
@diegoreymendez diegoreymendez requested a review from ayoy August 21, 2023 10:39
@diegoreymendez diegoreymendez self-assigned this Aug 21, 2023
Copy link
Collaborator

@ayoy ayoy left a comment

Choose a reason for hiding this comment

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

LGTM, all tests are passing locally for both targets for me 👍

@diegoreymendez diegoreymendez merged commit 7ba2300 into develop Aug 21, 2023
@diegoreymendez diegoreymendez deleted the diego/fix-waitlist-beta-unit-tests-creating-file branch August 21, 2023 10:56
samsymons added a commit that referenced this pull request Aug 22, 2023
# By Diego Rey Mendez (3) and others
# Via Diego Rey Mendez (1) and GitHub (1)
* develop:
  Require Duck Player scheme URL to be passed from YouTube Overlay User Script (#1519)
  extend valid urls to include IPv6 (#1492)
  Fix bug where Back navigation doesn't work properly with bangs (#1510)
  remove logic to keep previous search mode (#1500)
  Bump version to 1.53.0 (52)
  Update embedded files
  Adds a file that's generated when the unit tests are run (#1511)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants