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

Add ExtensionKit support #3109

Merged

Conversation

congt
Copy link
Contributor

@congt congt commented Nov 15, 2024

ExtensionKit extensions uses EXAppExtensionAttributes in its Info.plist. extension_point_identifiers_parser.py didn't support that so that it fails to find the ExtensionKit extensions.

Also adds an ExtensionKit extension to the example iOSAPP.

@congt congt requested a review from a team as a code owner November 15, 2024 17:59
@congt congt force-pushed the cshi/fix-extensionkit branch 2 times, most recently from 5a4fcb9 to e9fe324 Compare November 15, 2024 18:38
@congt congt changed the title Adds ExtensionKit support Add ExtensionKit support Nov 15, 2024
@congt congt force-pushed the cshi/fix-extensionkit branch from e9fe324 to bf920d1 Compare November 16, 2024 00:48
@congt congt force-pushed the cshi/fix-extensionkit branch from bf920d1 to 018de94 Compare November 16, 2024 01:25
ExtensionKit extensions uses EXAppExtensionAttributes in its info.plist. extension_point_identifiers_parser.py didn't support that so that it fails to find the ExtensionKit extensions.
Also adds an ExtensionKit extension to the example iOSAPP.

Signed-off-by: Cong Shi <[email protected]>
@congt congt force-pushed the cshi/fix-extensionkit branch from 018de94 to 7e0d0b7 Compare November 19, 2024 19:01
@brentleyjones brentleyjones enabled auto-merge (squash) November 19, 2024 19:03
@congt
Copy link
Contributor Author

congt commented Nov 26, 2024

Friendly ping for check runs again. The merge was blocked because the branch was out of date with the base branch.

@brentleyjones brentleyjones merged commit 53036fa into MobileNativeFoundation:main Nov 26, 2024
15 of 16 checks passed
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