apache / cordova-common

Apache Cordova Common Tooling Library
https://cordova.apache.org/
Apache License 2.0
39 stars 46 forks source link

feat(spm): Add method to determine if a plugin is a Swift package #216

Closed dpogue closed 7 hours ago

dpogue commented 1 week ago

Platforms affected

iOS

Motivation and Context

We want a declarative way for a plugin to opt-in to using Package.swift for managing its code and resources as opposed to file copying with source-file/header-file/resource-file/etc.

Description

Adds a helper method to detect if there is a <platform name="ios" package="swift"> tag.

Caveats:

Testing

Added unit test case.

Checklist

codecov-commenter commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.42%. Comparing base (24f7fa2) to head (5005e3f).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #216 +/- ## ========================================== + Coverage 88.40% 88.42% +0.02% ========================================== Files 20 20 Lines 1147 1149 +2 ========================================== + Hits 1014 1016 +2 Misses 133 133 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

dpogue commented 7 hours ago

I found a way to handle this in the platform that doesn't involve modifying cordova-common, so I'm going to close this