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 supported Xcode version to 16.0 #1001

Closed
wants to merge 38 commits into from
Closed

Update supported Xcode version to 16.0 #1001

wants to merge 38 commits into from

Conversation

ayoy
Copy link
Contributor

@ayoy ayoy commented Sep 20, 2024

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/1203301625297703/1208329354262556/f
iOS PR: duckduckgo/iOS#3377
macOS PR: duckduckgo/macos-browser#3320
What kind of version bump will this require?: Patch

Description:

This change updates CI workflows to use Xcode 16 and run on macOS 15

Steps to test this PR:

  1. Verify that CI is green

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16
  • macOS 10.15
  • macOS 11
  • macOS 12

Internal references:

Software Engineering Expectations
Technical Design Template

@ayoy ayoy marked this pull request as draft September 20, 2024 09:15
@ayoy ayoy marked this pull request as ready for review September 25, 2024 19:27
Copy link

github-actions bot commented Oct 3, 2024

This PR has been inactive for more than 7 days and will be automatically closed 7 days from now.

@github-actions github-actions bot added the stale label Oct 3, 2024
@github-actions github-actions bot removed the stale label Oct 9, 2024
ayoy and others added 13 commits October 14, 2024 10:13
* main:
  Second macOS CPM experiment (#1017)
  fix HStack width (#1022)
  Change tab URL calculation for contentblockerrules.js and surrogates.js (#1021)
  Add Events Firing for Phishing Detection Settings (#1012)
  Sabrina/onboarding UI updates (#1015)
* main:
  Use cookie to share subscription access token on DDG domains (#1034)
  update css dependency (#1037)
  Email parsing improved (#1016)
  Add NTP Search Bar feature flag (#1023)
  VPN logs fix (#1030)
  Add AI Chat feature flag (#1031)
  fix breakByRaisingSigInt mistakenly called when run w/o debugger (#1029)
samsymons and others added 15 commits October 29, 2024 21:53
* main:
  Validate VPN errors before re-throwing them (#1054)
  Allowing users to delete suggestions (#1027)
  Revert "Bump github.com/1024jp/gzipswift from 6.0.1 to 6.1.0" (#1055)
  Bump github.com/duckduckgo/privacy-dashboard from 5.3.0 to 7.1.1 (#1046)
  Bump github.com/duckduckgo/sync_crypto from 0.2.0 to 0.3.0 (#1048)
  Bump github.com/1024jp/gzipswift from 6.0.1 to 6.1.0 (#1050)
* main:
  Update C-S-S to 6.29.0 (#1062)
  Send pixel on sync secure storage read failure (#1058)
* main:
  Remove test rollout feature (#1061)
  VPN clean-up (#1041)
  Deprecate PixelKit daily pixel suffixes (#1060)
These changes should be cleared with Thom first, and are unrelated to Xcode 16.
* main:
  Make sure to apply only relevant (last requested) rules on attribution (#1059)
  Update TLD list  (#1018)
* main:
  Remediate TunnelVision, TunnelCrack and fix "Exclude Local Networks" (#1039)
  Sync: Send pixels for account removal + decoding issues (#1065)
  Bump github.com/duckduckgo/content-scope-scripts from 6.31.0 to 6.32.0 (#1067)
  Behavioral toast updates (#1063)
  point to CSS branch (#1051)
Comment on lines -414 to -417
resources: [
.copy("hashPrefixes.json"),
.copy("filterSet.json")
],
Copy link
Contributor

Choose a reason for hiding this comment

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

I checked this with @not-a-rootkit, this change is trying to copy files which don't exist on-disk which causes problems with Xcode 16 - we can safely remove them.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, these warnings were becoming annoying :)

Comment on lines +182 to +188
let configDataExpectation = XCTNSPredicateExpectation(predicate: NSPredicate { _, _ in
return managerA.dependencyProvider.privacyConfigData == configData
}, object: .none)

let configEtagExpectation = XCTNSPredicateExpectation(predicate: NSPredicate { _, _ in
return managerA.dependencyProvider.privacyConfigEtag == HTTPURLResponse.testEtag
}, object: .none)
Copy link
Contributor

Choose a reason for hiding this comment

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

This test has a race condition related to file coordinators that has since become more obvious with Xcode 16. This change to wait for the expected value has fixed it 100% of the time for me, even after hundreds of runs.

* main:
  implement experiment manager (#1066)
@samsymons samsymons closed this Nov 13, 2024
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.

3 participants