Summary:
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] describe the change (for example, what happens before the change, and after the change)
This is a step towards aligning Swift and Objective-C into a single SDK.
Currently we cannot ship a single target that includes both Swift and Objective-C because some consumers of the SDK are tied to older versions of Xcode. Including Swift would increase their binary size or force them to upgrade before Apple mandates an upgrade.
What we can do is prepare for this inevitability by moving the Swift files into this repository.
Additionally this is easier to maintain and allows us to us to reference ObjC files from Swift directly instead of treating them as a third party dependency.
This work raises a few questions about distribution channels (what will the starting versions be for instance) but that is an acceptable cost for the benefits we get from having all of our related code in one place.
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/1031
Test Plan:
Able to build is the primary concern. That the old targets are not impacted by the new targets and that the new targets build correctly.
Ensure the older targets can build on Xcode 10.1
* FBSDKCoreKit
* FBSDKShareKit
* FBSDKLoginKit
Ensure the older AND newer targets can build on Xcode 10.3
* FBSDKCoreKit
* FBSDKShareKit
* FBSDKLoginKit
* FBSDKCoreKitSwift
* FBSDKShareKitSwift
* FBSDKLoginKitSwift
Ensure the older AND newer targets can build on the latest Xcode (11 beta 6)
// Note: Will need to clean build if you want to build the non-Swift version after building the Swift version
* FBSDKCoreKit
* FBSDKShareKit
* FBSDKLoginKit
* FBSDKCoreKitSwift
* FBSDKShareKitSwift
* FBSDKLoginKitSwift
Reviewed By: dreamolight
Differential Revision: D17164409
Pulled By: joesus
fbshipit-source-id: dbda5036487a2e626e271651b38262e56dac17ce
Summary:
With the introduction of the `FBSDKCoreKit.modulemap` to the Basics folder, after updating to the newer version of the pod, I keep getting the following warning:
<img width="470" alt="Screen Shot 2019-05-01 at 5 51 06 pm" src="https://user-images.githubusercontent.com/4775087/57009482-384d3e00-6c3a-11e9-95cb-80ed1ed3e795.png">
As it is not a source file, Xcode doesn't need to analyse because it won't be able to compile it.
- Restricted the type of source files to `.h` and `.m`
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [x] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [x] describe the change (for example, what happens before the change, and after the change)
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/988
Reviewed By: codytwinton
Differential Revision: D15205123
Pulled By: tianqibt
fbshipit-source-id: 1fbfff432e25a7906589be7f209200f48b3f1601
Summary: Create subspec for Basics and make other parts in CoreKit depends on Basics.
Reviewed By: dreamolight
Differential Revision: D14875453
fbshipit-source-id: dbace790817eb2031f59ef47a30a6765da9a7da6
Summary:
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [ ] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [ ] describe the change (for example, what happens before the change, and after the change)
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/981
Reviewed By: jingping2015
Differential Revision: D14766112
Pulled By: codytwinton
fbshipit-source-id: 006aceb137881415f47eb1aa2ef2f2ef7879a3df
Summary: Bumped version for first 5.0 RC
Reviewed By: dreamolight
Differential Revision: D14718312
fbshipit-source-id: 91522df58a87efd015f3458f506ba052310e8a39
Summary: Retroactively bumped version to match 4.x-branch
Reviewed By: jingping2015
Differential Revision: D14718291
fbshipit-source-id: aeaec62a664c29f5c3004e421771f160a189b27c
Summary:
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [ ] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [ ] describe the change (for example, what happens before the change, and after the change)
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/968
Reviewed By: joesus
Differential Revision: D14553051
Pulled By: codytwinton
fbshipit-source-id: a44efd422bbc785a6a5bbd8fd66700b5e707de16
Summary:
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [X] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [X] describe the change (for example, what happens before the change, and after the change)
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/965
Reviewed By: joesus
Differential Revision: D14514474
Pulled By: codytwinton
fbshipit-source-id: 0848c8bb63004c739b1734a04d20ed82371d2874
Summary:
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/963
- As title
- Move `FBSDKCoreKit+Internal.h` back to `Internal` folder as cocoapods doesn't support excluding files in `public_header_files`
- Update project files due to above change for other kits
Reviewed By: joesus, codytwinton
Differential Revision: D14494473
fbshipit-source-id: e37b7507557c6f9a6590163afd3e33f52b2c0de2
Summary:
Thanks for proposing a pull request.
To help us review the request, please complete the following:
- [X] sign [contributor license agreement](https://developers.facebook.com/opensource/cla)
- [X] describe the change (for example, what happens before the change, and after the change)
Pull Request resolved: https://github.com/facebook/facebook-objc-sdk/pull/956
Differential Revision: D14465436
Pulled By: codytwinton
fbshipit-source-id: 6b8894e658f9c27ac6e56cf3ea4d347d01f597ca
Summary:
The Bolts framework served a minor role in this SDK, but maintaining this dependency was a decent pain point.
This diff removes the dependency on this framework.
Reviewed By: jingping2015
Differential Revision: D10440950
fbshipit-source-id: 58d78f9989ff394b3b29bf39314699cd31ba4898
Summary: Changed release tag format from `sdk-version-0.0.0` to `v0.0.0`
Reviewed By: jingping2015
Differential Revision: D13524844
fbshipit-source-id: 323663ef5ec91c1c3c2682cd05de56d1dcd30254
Summary: This resolves some link issues
Reviewed By: jingping2015
Differential Revision: D10096232
fbshipit-source-id: 14644e7e9595d87316c9b606ada3253d13e0946e
Summary: The GitHub repo has been renamed from facebook-ios-sdk to facebook-objc-sdk
Reviewed By: kalendae
Differential Revision: D9833826
fbshipit-source-id: f3e45918656ed2e0b16e872c34717974bfbf64dd
