Skip to content

[c++-interop] APINotes test for ObjCMethods #41998

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

Merged
merged 1 commit into from
Apr 4, 2022
Merged

Conversation

plotfi
Copy link
Contributor

@plotfi plotfi commented Mar 24, 2022

This is swift test to go with swiftlang/llvm-project#4118

The test is verifying that a ObjCMethod that is annotated by an APINote
is in fact getting the APINote applied. These types of APINotes are
heavily used by UIKit for instance in didMoveToParentViewController in
UIViewController.

This is swift test to go with swiftlang/llvm-project#4118

The test is verifying that a ObjCMethod that is annotated by an APINote
is in fact getting the APINote applied. These types of APINotes are
heavily used by UIKit for instance in didMoveToParentViewController in
UIViewController.
@plotfi
Copy link
Contributor Author

plotfi commented Mar 24, 2022

@drodriguez
Copy link
Contributor

swiftlang/llvm-project#4118

@swift-ci please smoke test

@plotfi
Copy link
Contributor Author

plotfi commented Mar 31, 2022

swiftlang/llvm-project#4118

@swift-ci please smoke test

@plotfi
Copy link
Contributor Author

plotfi commented Mar 31, 2022

@swift-ci please smoke test

@plotfi
Copy link
Contributor Author

plotfi commented Mar 31, 2022

Going to merge this soon since @zoecarver's swiftlang/llvm-project#4148 would make this test pass.

@plotfi
Copy link
Contributor Author

plotfi commented Mar 31, 2022

@swift-ci smoke test macos

2 similar comments
@plotfi
Copy link
Contributor Author

plotfi commented Mar 31, 2022

@swift-ci smoke test macos

@plotfi
Copy link
Contributor Author

plotfi commented Apr 4, 2022

@swift-ci smoke test macos

@plotfi plotfi merged commit 338af18 into swiftlang:main Apr 4, 2022
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.

3 participants