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

Skip /Applications symlink script for Debug builds #1443

Merged
merged 1 commit into from
Aug 7, 2023

Conversation

ayoy
Copy link
Collaborator

@ayoy ayoy commented Aug 7, 2023

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

Description:
Run the /Applications symlink script only on Debug builds because it has unexpected
and severe consequences when run in Release configuration (such as deleting the
production release app bundle).

Steps to test this PR:
Verify that these workflow runs are passing:


Internal references:

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

Copy link
Collaborator

@samsymons samsymons left a comment

Choose a reason for hiding this comment

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

Really nice find 👏

@samsymons samsymons assigned ayoy and unassigned samsymons Aug 7, 2023
@ayoy ayoy merged commit 27be6d5 into develop Aug 7, 2023
18 of 21 checks passed
@ayoy ayoy deleted the dominik/gha-fix-release-workflow branch August 7, 2023 22:51
loremattei pushed a commit that referenced this pull request Aug 8, 2023
Task/Issue URL: https://app.asana.com/0/1203301625297703/1205226515221343/f

Description:
Run the /Applications symlink script only on Debug builds because it has unexpected
and severe consequences when run in Release configuration (such as deleting the
production release app bundle).
samsymons added a commit that referenced this pull request Aug 13, 2023
# By Dominik Kapusta (5) and others
# Via GitHub (1) and Lorenzo Mattei (1)
* develop:
  Add support for syncing Credentials (#1368)
  Fix dsym upload to S3
  Bump version to 1.51.1 (48)
  Report failed unit tests to Asana (#1457)
  Use MainActor to ensure FutureExtension tests are dispatched on main queue (#1472)
  Move Make /Applications symlink build phase to later (#1470)
  don't show day 7 when day 0 has been dismissed (#1468)
  Update BSK with autofill 8.1.2 (#1471)
  Add job to create a task on asana if tests fails on develop (#1461)
  Disables a flaky test from App Store unit tests (#1467)
  Fix an autoconsent exception on old Safari (#1460)
  fix bug where wrong address bar text is shown when opening DuckPlayer… (#1434)
  Fix FutureExtensionTests (#1450)
  Add pixel to determine Bitwarden use on Mac (#1412)
  Skip /Applications symlink script for Debug builds (#1443)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
samsymons added a commit that referenced this pull request Aug 13, 2023
* develop:
  Update NetP endpoint (#1459)
  Add support for syncing Credentials (#1368)
  Fix dsym upload to S3
  Bump version to 1.51.1 (48)
  Report failed unit tests to Asana (#1457)
  Use MainActor to ensure FutureExtension tests are dispatched on main queue (#1472)
  Move Make /Applications symlink build phase to later (#1470)
  don't show day 7 when day 0 has been dismissed (#1468)
  Update BSK with autofill 8.1.2 (#1471)
  Add job to create a task on asana if tests fails on develop (#1461)
  Disables a flaky test from App Store unit tests (#1467)
  Fix an autoconsent exception on old Safari (#1460)
  fix bug where wrong address bar text is shown when opening DuckPlayer… (#1434)
  Fix FutureExtensionTests (#1450)
  Add pixel to determine Bitwarden use on Mac (#1412)
  Encapsulate NetP subjects within observers (#1436)
  Skip /Applications symlink script for Debug builds (#1443)
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