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

Support for Xcode String Catalogs #931

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

mbernson
Copy link
Collaborator

@mbernson mbernson commented Feb 7, 2025

This PR adds support for Xcode String Catalogs to R.swift.

The string catalogs are parsed in a similar fashion to Strings files and Strings Dicts. They output the same type, a StringsTable.

What's changed in this PR

  • Extended StringsTable+Parser.swift with support for parsing string catalogs.
  • Added unit tests for the StringsTable parser under RswiftParsersTests.
  • Added a test app StringCatalogApp as an integration test. This is also used in the checks workflow on GitHub Actions.

Fixes #840, #861

@mbernson mbernson requested a review from tomlokhorst February 7, 2025 13:51
@mbernson
Copy link
Collaborator Author

mbernson commented Feb 7, 2025

Of course, after creating this PR I noticed that #886 also exists with an implementation of this. 🤦🏼

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.

Support xcstring on Xcode 15
1 participant