Skip to content

Add Swift Package Manager (SPM) support for several Foundation targets #540

Merged
thomasvl merged 1 commit intogoogle:mainfrom
bc-lee:spm-init
Sep 19, 2025
Merged

Add Swift Package Manager (SPM) support for several Foundation targets #540
thomasvl merged 1 commit intogoogle:mainfrom
bc-lee:spm-init

Conversation

@bc-lee
Copy link
Copy Markdown
Contributor

@bc-lee bc-lee commented Aug 1, 2025

This PR introduces Swift Package Manager support for the following targets:

  • GTMDefines
  • GTMLogger
  • GTMNSData_zlib
  • GTMStringEncoding

Test targets for each of these have also been set up, based on the existing source code structure. To better support SPM, the UnitTesting/SenTestCase directory was created, and several test-supporting files were moved into it from the UnitTesting directory.

A new GitHub Actions workflow (spm.yml) has been added to ensure continuous integration for the SPM setup.

Additional targets will be integrated in future iterations as needed.
Related: #258

Comment thread Sources/Foundation/Logger/GTMLogger.m Outdated
@bc-lee bc-lee force-pushed the spm-init branch 2 times, most recently from 5c05663 to 94e47a6 Compare September 9, 2025 21:25
@bc-lee bc-lee marked this pull request as ready for review September 9, 2025 21:30
@bc-lee bc-lee requested a review from thomasvl September 9, 2025 21:30
Comment thread Sources/Defines/include/GTMDefines.h Outdated
Comment thread UnitTesting/SenTestCase/GTMSenTestCase.h
Comment thread .github/workflows/spm.yml
Comment thread .github/workflows/spm.yml
@bc-lee bc-lee requested a review from thomasvl September 10, 2025 18:37
@bc-lee
Copy link
Copy Markdown
Contributor Author

bc-lee commented Sep 18, 2025

Thanks! This should be the final change in this series. Could you take a look once more?

@thomasvl
Copy link
Copy Markdown
Member

Thanks for doing all the work to get to this!

@thomasvl thomasvl merged commit 521a835 into google:main Sep 19, 2025
11 checks passed
@bc-lee bc-lee deleted the spm-init branch September 19, 2025 15:04
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.

2 participants