Skip to content

Conversation

@damien-rivet
Copy link
Contributor

Resolves #919

Short description 📝

This PR adds the missing Global Accent Color Name (ASSETCATALOG_COMPILER_GLOBAL_ACCENT_COLOR_NAME) build setting for App targets.

Solution 📦

I followed the trail of ASSETCATALOG_COMPILER_APPICON_NAME to find where to add the missing build setting.

Implementation 👩‍💻👨‍💻

  • Add ASSETCATALOG_COMPILER_GLOBAL_ACCENT_COLOR_NAME to App default build settings
  • Refactor BuildSettingsProvider tests with Swift Testing

@dosubot dosubot bot added the size:XL This PR changes 500-999 lines, ignoring generated files. label Apr 1, 2025
Copy link
Member

@fortmarek fortmarek left a comment

Choose a reason for hiding this comment

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

Thanks! This looks good to me 🙂

We can merge once the CI agrees 😁

struct BuildSettingProviderTests {

@Test("Target settings", arguments: [
TestParameters(
Copy link
Member

Choose a reason for hiding this comment

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

This is much more readable 🎉

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 1, 2025
@damien-rivet damien-rivet force-pushed the fix/919-add-missing-global-accent-color-name-build-setting-for-apps branch from f519d3d to be7d9da Compare April 7, 2025 09:47
@fortmarek fortmarek merged commit 99fa352 into tuist:main Apr 7, 2025
6 checks passed
@damien-rivet damien-rivet deleted the fix/919-add-missing-global-accent-color-name-build-setting-for-apps branch April 28, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Global Accent Color Name build setting is missing from target settings

3 participants