Skip to content

fix: remove incorrect warn about icon 1024x1024 size#3

Merged
keiver merged 1 commit into
mainfrom
fix-warn-size-1024
Feb 1, 2026
Merged

fix: remove incorrect warn about icon 1024x1024 size#3
keiver merged 1 commit into
mainfrom
fix-warn-size-1024

Conversation

@keiver

@keiver keiver commented Feb 1, 2026

Copy link
Copy Markdown
Owner

This pull request updates the minimum and recommended requirements for icon images in the TvOS asset processing tool. The changes remove the recommended icon size threshold (1280x1280), making 1024x1024 both the minimum and sufficient size for all outputs. Related warnings and tests for the recommended icon size have also been removed for consistency.

Icon size requirement updates:

  • The recommended icon size (1280x1280) has been removed from the documentation in README.md; 1024x1024 is now considered sufficient for all outputs.
  • The ICON_RECOMMENDED constant has been removed from src/config.ts, and all references to a recommended icon size have been deleted.

Validation logic and test updates:

  • The warning for icon images below the recommended size has been removed from the validation logic in src/config.ts.
  • The corresponding test that checked for a warning when the icon was below the recommended size has been removed from tests/config.test.ts.
  • The comment for createTestIcon in tests/fixtures/create-fixtures.ts has been updated to remove reference to the recommended minimum.

@keiver keiver self-assigned this Feb 1, 2026
@keiver keiver added the version:patch 1.0.x label Feb 1, 2026
@keiver
keiver merged commit 7fefe15 into main Feb 1, 2026
2 checks passed
@github-actions

github-actions Bot commented Feb 1, 2026

Copy link
Copy Markdown
Contributor

Released as v1.2.1 and published to npm.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant