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

Move away from resource helper. #131

Closed
wants to merge 6 commits into from

Conversation

Diggory
Copy link
Contributor

@Diggory Diggory commented Sep 7, 2023

Updates swift tools version in manifest to 5.5
Moves test sample files into their own directory.
Removes ResourceHelper and uses built-in function for getting resources as offered by SPM since 5.5
Fixes #130

@Diggory Diggory changed the title Move away from resource helper. Fixes #130 Move away from resource helper. Sep 7, 2023
@DivineDominion
Copy link
Contributor

The CSV file copy phase looks nice!

Could you double check that the Xcode Project in the SwiftCSV still compiles and tests? I believe you need to switch between Bundle.module for SPM and Bundle.init for Carthage, direct build, etc., see: #130 (comment)

@Diggory
Copy link
Contributor Author

Diggory commented Sep 18, 2023

Ah! I didn't check the Xcode project. I now have a separate branch which solves that (and brings back ResourceHelper.)

I will close this PR and generate another one from that branch.
Thanks!

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.

Tests fail in Xcode due to ResourceHelper using wrong path
2 participants