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

10.14 Build Target #198

Draft
wants to merge 13 commits into
base: develop
Choose a base branch
from

Conversation

jncn
Copy link
Collaborator

@jncn jncn commented Dec 15, 2020

The PR for #197

…into develop

# Conflicts:
#	SubEthaEdit-Mac/Source/OpenURLViewController.h
#	SubEthaEdit-Mac/Source/OpenURLViewController.m
#	SubEthaEdit-Mac/SubEthaEdit.xcodeproj/project.pbxproj
#	SubEthaEdit-Mac/de.lproj/OpenURLViewController.xib
…into develop

# Conflicts:
#	SubEthaEdit-Mac/Source/SEEWebPreviewViewController.m
Reduced warnings from 230+ to 48
[sharingServices removeObject:[NSSharingService sharingServiceNamed:NSSharingServiceNamePostOnTwitter]];
[sharingServices removeObject:[NSSharingService sharingServiceNamed:NSSharingServiceNamePostOnSinaWeibo]];
[sharingServices removeObject:[NSSharingService sharingServiceNamed:NSSharingServiceNamePostOnTencentWeibo]];
[sharingServices removeObject:[NSSharingService sharingServiceNamed:NSSharingServiceNamePostOnLinkedIn]];
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

These are not just deprecated: from 10.14 these services are not longer present

@monkeydom
Copy link
Contributor

You still have your url changes in the change request - probably need to rebase/change your baseline to develop at some point in time ^^

…ndle version change, moved deployment target change to project level
@monkeydom monkeydom linked an issue Dec 16, 2020 that may be closed by this pull request
@monkeydom monkeydom added this to the v5.2 milestone Dec 16, 2020
@monkeydom monkeydom modified the milestones: v5.2, v5.3 Mar 2, 2021
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.

Set minimum build target to 10.14
2 participants