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.

Sign up for GitHub

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

Jump to bottom

Fix unhandled file warning when building with SPM #2132

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Fix unhandled file warning when building with SPM #2132

merged 1 commit into from
Aug 10, 2023

Conversation

calda
Copy link
Member

@calda calda commented Aug 8, 2023

This PR fixes an "unhandled file" warning I noticed when running swift build

$ swift build
warning: 'lottie': found 3 file(s) which are unhandled; explicitly declare them as resources or exclude from the target
    /Users/calstephens/Documents/lottie/Sources/Private/EmbeddedLibraries/EpoxyCore/README.md
    /Users/calstephens/Documents/lottie/Sources/Private/EmbeddedLibraries/README.md
    /Users/calstephens/Documents/lottie/Sources/Private/EmbeddedLibraries/ZipFoundation/README.md

@calda calda force-pushed the cal--resolve-unhandled-file-warning branch from af454df to 8b4708a Compare August 8, 2023 21:08
@calda calda force-pushed the cal--resolve-unhandled-file-warning branch from 8b4708a to be8aa94 Compare August 10, 2023 15:47
@calda calda enabled auto-merge (squash) August 10, 2023 15:49
@calda calda merged commit 8b89a43 into master Aug 10, 2023
12 checks passed
@calda calda deleted the cal--resolve-unhandled-file-warning branch August 10, 2023 16:18
@calda calda mentioned this pull request Aug 24, 2023
Closed
iago849 pushed a commit to atteamapps/lottie-ios that referenced this pull request Feb 8, 2024
@calda @iago849
Fix unhandled file warning when building with SPM (airbnb#2132)
c9fce4b
MoroziOS pushed a commit to MoroziOS/tmg-lottie-ios that referenced this pull request May 22, 2024
@calda
Fix unhandled file warning when building with SPM (airbnb#2132)
08c8370
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miguel-jimenez-0529 miguel-jimenez-0529 miguel-jimenez-0529 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@calda @miguel-jimenez-0529