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

Remove the ld_classic build setting #3575

Merged
merged 1 commit into from
Nov 14, 2024
Merged

Conversation

samsymons
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/1199333091098016/1208760941595237/f
Tech Design URL:
CC:

Description:

This PR removes the legacy linker setting.

Steps to test this PR:

  1. Check that CI is green
  2. Check that the app runs as expected on a local device

Definition of Done (Internal Only):

Copy Testing:

  • Use of correct apostrophes in new copy, ie rather than '

Orientation Testing:

  • Portrait
  • Landscape

Device Testing:

  • iPhone SE (1st Gen)
  • iPhone 8
  • iPhone X
  • iPhone 14 Pro
  • iPad

OS Testing:

  • iOS 15
  • iOS 16
  • iOS 17

Theme Testing:

  • Light theme
  • Dark theme

Internal references:

Software Engineering Expectations
Technical Design Template

@samsymons samsymons requested review from a team, jdjackson, studiosutara and afterxleep and removed request for a team, jdjackson and studiosutara November 13, 2024 23:53
Copy link
Collaborator

@afterxleep afterxleep left a comment

Choose a reason for hiding this comment

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

LGTM, build & run on device, all good.

@samsymons
Copy link
Contributor Author

The Asana subtask check failed but all others succeeded, merging this.

@samsymons samsymons merged commit 81c9a09 into main Nov 14, 2024
28 of 31 checks passed
@samsymons samsymons deleted the sam/migrate-to-new-linker branch November 14, 2024 16:36
samsymons added a commit that referenced this pull request Nov 14, 2024
# By Daniel Bernal (3) and others
# Via GitHub
* main:
  Remove the `ld_classic` build setting (#3575)
  text zoom improvements (#3509)
  Add support for local overrides for feature flags (#3571)
  pixels for bookmarks > favorites tab (#3572)
  Sync BSK with macos (#3561)
  Add possibility to stop page loading (#3553)
  [DuckPlayer] Overlay Usage Pixels (#3565)
  Update pr.yml
  Update pr.yml

# 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