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

build: add xcconfig, to simplify version updates #250

Merged
merged 8 commits into from
Oct 18, 2024

Conversation

andrewjl-mux
Copy link
Contributor

@andrewjl-mux andrewjl-mux commented Oct 16, 2024

  • Add xcconfig for each target, to simplify version updates
  • Remove iOS 11 deployment target flags that aren't applicable (and would result in build failures on Xcode 15+ if they were)
  • Add Xcode path log to static build script for consistency
  • Skip SPM and Cocoapods tests on non-release branches

dispatch a viewend automatically and tear down player
monitoring when the application shuts down
@andrewjl-mux andrewjl-mux requested a review from a team as a code owner October 16, 2024 21:37
Copy link
Collaborator

@daytime-em daytime-em left a comment

Choose a reason for hiding this comment

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

my only regret is that i have but one ✅ to give

@andrewjl-mux andrewjl-mux changed the base branch from ajlb/fix/tear-down-view-on-hard-application-termination to releases/v4.1.1 October 18, 2024 20:53
@andrewjl-mux andrewjl-mux merged commit 7e7b1ba into releases/v4.1.1 Oct 18, 2024
3 checks passed
@andrewjl-mux andrewjl-mux deleted the ajlb/build/add-xcconfig branch October 18, 2024 20:53
@andrewjl-mux andrewjl-mux mentioned this pull request Oct 18, 2024
andrewjl-mux added a commit that referenced this pull request Oct 18, 2024
* fix: tear down view on hard application termination

dispatch a viewend automatically and tear down player
monitoring when the application shuts down

* build: add xcconfig, to simplify version updates

* build: use per target xcconfig to match core

* ci: consistent logs

* ci: make these scripts consistent

* fix indentation

* build: add comments on config names

* ci: skip lengthy app tests when not on release branch
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