Make EventKit Imports Explicit

[forgot]

Release 1.5.8 (#138) exposes EventKit in all targets through their dependency on the CorePermissionsSwiftUI module. This causes all target libraries to reference EventKit, even if they don't use it.

This PR:

• Adds a SwiftSetting defining a PERMISSIONSWIFTUI_EVENT flag to the target definitions for JMCalendarPermissionManager and JMRemindersPermissionManager
• Wraps EventPermissionManager, JMRemindersPermissionManager, and JMCalendarPermissionManager with a compile time check for PERMISSIONSWIFTUI_EVENT
• Fixes [BUG] - ITMS-90683: Missing purpose string in Info.plist #140

[jevonmao]

Thanks!

[jevonmao]

@forgot Have you tested these changes locally? I didn't check when merging the PR, but now I seem to be getting many Xcode errors from Reminder and Calendar managers when building.

[forgot]

Yes, and all tests were passing after the changes. I've even used the fork to deploy to the AppStore with no issues, but I am only using the tracking permission currently.

I used the same method that was used to isolate HealthKit, where a flag is defined in the Swift Package and checked when any modules using EventKit are used.

Are you only getting errors when building, or are the tests failing as well? I'm not at my computer now, but I'll take another look when I get back.

[jevonmao]

I was getting errors by building. I'm not familiar with certain behaviors in SPM flags trying to debug, I ended up doing a quick fix by moving the EventPermissionManager into its own target, so all good now.

The problem should be resolved because EventKit is no longer referenced in CorePermissionsSwiftUI. Let me know I missed anything