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

Increase Deployment Target to iOS 13 #2459

Merged
merged 14 commits into from
Jan 3, 2025
Merged

Conversation

zeitschlag
Copy link
Collaborator

@zeitschlag zeitschlag commented Dec 22, 2024

This PR increases the deployment target to iOS 13. It fixes most warnings (except #2471, which can be done in a subsequent PR, as discussed), removes @availables (in our code),

Oh, the overall LOC increased as the pods got updated, so please feel free to ignore that entire folder.

@zeitschlag zeitschlag force-pushed the 2453-ios-12-byyyyyyeeeeee branch 3 times, most recently from 2b68ab3 to e8edcc1 Compare January 3, 2025 13:51
@zeitschlag zeitschlag force-pushed the 2453-ios-12-byyyyyyeeeeee branch from e8edcc1 to 4679929 Compare January 3, 2025 14:19
@zeitschlag zeitschlag changed the title [WIP] ios 12 byyyyyyeeeeee Increase Deployment Target to iOS 13 Jan 3, 2025
@zeitschlag zeitschlag marked this pull request as ready for review January 3, 2025 15:54
@zeitschlag zeitschlag requested a review from r10s January 3, 2025 15:56
@zeitschlag zeitschlag self-assigned this Jan 3, 2025
@r10s
Copy link
Member

r10s commented Jan 3, 2025

getting the following error on building:

Screenshot 2025-01-03 at 17 09 42

@zeitschlag
Copy link
Collaborator Author

zeitschlag commented Jan 3, 2025

getting the following error on building:

As both me and the CI can build that thing without getting that error I'd assume that there's something off with your local project. Have you tried CMD+Shift+King?

Usually that build error indicates that the file is still part of Compile Sources in Build Phases, but I doublechecked: It's not there.

@r10s
Copy link
Member

r10s commented Jan 3, 2025

thanks for the hint, i will double-check, code-wise, this lgtm, so we can merge in

i meanwhile also created deltachat/deltachat-pages#1023 to update the requirements shown on https://delta.chat

@zeitschlag zeitschlag merged commit 3ad676f into main Jan 3, 2025
1 check passed
@zeitschlag zeitschlag deleted the 2453-ios-12-byyyyyyeeeeee branch January 3, 2025 16:33
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