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

Use fastlane match for code signing DMG builds #3246

Merged
merged 7 commits into from
Sep 11, 2024

Conversation

ayoy
Copy link
Collaborator

@ayoy ayoy commented Sep 10, 2024

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

Description:
This change adds support for using fastlane match to manage code signing with
Developer ID Application certificate. New lanes are added for syncing code signing
for local and CI use. Build settings are updated to use provisioning profiles generated
by fastlane. archive.sh script and all CI workflows are updated to use fastlane for signing
DMG builds and other CI binaries that use Developer ID Application certificate (e.g. unit tests).

Steps to test this PR:

  1. Verify that the notarized review DMG workflow completed successfully.
  2. Verify that the notarized release DMG workflow completed successfully.
  3. Verify that the UI tests workflow runs tests as expected (macOS 14 tests are failing atm for an unrelated reason).
  4. Verify that the Sync E2E tests workflow completed successfully.
  5. Verify that the variants workflow completed successfully.

Definition of Done:


Internal references:

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

Copy link
Contributor

github-actions bot commented Sep 10, 2024

Warnings
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS against 3b3c966

@ayoy ayoy force-pushed the dominik/fastlane-match-developer-id branch from 8e5b82f to 1e2d8b0 Compare September 10, 2024 22:13
@ayoy ayoy changed the title Use fastlane match for code signing with Developer ID Application certificate Use fastlane match for code signing DMG builds Sep 10, 2024
@ayoy ayoy marked this pull request as ready for review September 10, 2024 22:38
@ayoy ayoy requested a review from diegoreymendez September 11, 2024 11:38
Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

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

After chatting with @ayoy, I'm approving this even though UI tests are failing. The changes introduced here can't have an effect on UI tests - I've seen these failing a lot.

@ayoy tells me work is being done to fix the UI tests so that's enough on my end.

@ayoy ayoy merged commit bb0f394 into release/1.106.0 Sep 11, 2024
206 of 208 checks passed
@ayoy ayoy deleted the dominik/fastlane-match-developer-id branch September 11, 2024 17:34
samsymons added a commit that referenced this pull request Sep 12, 2024
# By Dominik Kapusta (4) and others
# Via Dominik Kapusta (4) and GitHub (2)
* main:
  Support dragging strings to new tab button (#3228)
  BSK - Add feature flag for SKAN API (#3265)
  Ignore accents when doing a bookmark search (#3255)
  Update UI tests workflows to always compile the app on the supported version of Xcode (#3262)
  Bump C.S.S (via BSK) (#3244)
  Tab previews presentation implementation adjusted for the latest MouseOverView logic (#3260)
  Bump version to 1.106.0 (260)
  Default to appstore signing for fastlane match
  Clean up secrets in GHA workflows
  Update Onboarding gradients (#3248)
  Wait for UserScripts to be loaded for email protection (#3252)
  Fix UI tests failing in the CI (#3254)
  Fix WKDownloadMock mock (#3258)
  VPN Proxy stop and pixel improvements (#3239)
  Use fastlane match for code signing DMG builds (#3246)
  Implement new VPN site troubleshooting report alert (#3224)

# 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