Skip to content

SR-15910: C++ Interop Getting Started Documentation #41580

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

Conversation

cabmeurer
Copy link
Contributor

Create a document specific to getting started for utilizing C++ interop.
The examples provided are Add C++ to an xcode project, Creating a Swift Package, and Building with CMake.

@cabmeurer cabmeurer force-pushed the SR-15910-C++-Interop-Getting-Started-Documentation branch 3 times, most recently from 705448f to 0911e7d Compare February 28, 2022 17:53
@cabmeurer
Copy link
Contributor Author

here is the repo i made to show these examples: https://github.com/cabmeurer/CxxInterop_Examples

@zoecarver zoecarver requested review from hyp and compnerd February 28, 2022 20:41
@zoecarver zoecarver added the c++ interop Feature: Interoperability with C++ label Feb 28, 2022
@zoecarver
Copy link
Contributor

Thank you for the patch. This looks really great!

I have a few comments, but they're all pretty minor. This mostly looks good. Can't wait to land it.

Until you have commit access, you won't be able to trigger swift-ci. You can just ask one of us to do it. I will run the bots once you address my comments above :)

@cabmeurer cabmeurer force-pushed the SR-15910-C++-Interop-Getting-Started-Documentation branch 2 times, most recently from 419018b to 879671a Compare February 28, 2022 21:55
@cabmeurer cabmeurer requested a review from zoecarver February 28, 2022 21:57
Copy link
Contributor

@zoecarver zoecarver left a comment

Choose a reason for hiding this comment

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

Almost there. Looks great. Thanks again!

@cabmeurer cabmeurer force-pushed the SR-15910-C++-Interop-Getting-Started-Documentation branch 2 times, most recently from ec3c9fd to a08f76b Compare February 28, 2022 23:14
@cabmeurer cabmeurer requested a review from zoecarver February 28, 2022 23:15
@cabmeurer cabmeurer force-pushed the SR-15910-C++-Interop-Getting-Started-Documentation branch 3 times, most recently from 2a5e901 to 77b75a4 Compare February 28, 2022 23:29
Copy link
Contributor

@zoecarver zoecarver left a comment

Choose a reason for hiding this comment

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

One last thing. Once this is fixed I'll run the tests. Thanks for your patients. Looking forward to landing this!

@cabmeurer cabmeurer force-pushed the SR-15910-C++-Interop-Getting-Started-Documentation branch 2 times, most recently from 6dc023c to 2cab173 Compare February 28, 2022 23:32
@cabmeurer cabmeurer force-pushed the SR-15910-C++-Interop-Getting-Started-Documentation branch from 2cab173 to 7e89097 Compare February 28, 2022 23:36
@zoecarver
Copy link
Contributor

@swift-ci please smoke test.

@zoecarver zoecarver merged commit 417adb5 into swiftlang:main Mar 1, 2022
@cabmeurer cabmeurer deleted the SR-15910-C++-Interop-Getting-Started-Documentation branch March 1, 2022 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ interop Feature: Interoperability with C++
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants