Skip to content

Conversation

@KaustubhKulkarni2202
Copy link

  • Adds a privacy manifest.
  • Updates Package.swift to include the privacy manifest
  • Updates podspec to include the privacy manifest

@ZevEisenberg
Copy link
Collaborator

Thanks! Maybe this will be the push I need to get our CI working again. I have an old WIP PR at #433 that I might dust off.

@ZevEisenberg ZevEisenberg mentioned this pull request Mar 26, 2024
Kaustubh Kulkarni added 2 commits March 28, 2024 12:53
],
resources: [
.process("Resources"),
.copy("PrivacyInfo.xcprivacy"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is this needed in the test target?

Choose a reason for hiding this comment

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

No

Choose a reason for hiding this comment

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

not needed

),
exclude: ["Info.plist"],
resources: [
.copy("PrivacyInfo.xcprivacy"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why copy instead of process?

Choose a reason for hiding this comment

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

As we are not applying a platform-specific rule to the resource we need copy of it.

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