Skip to content

Align actool with upstream#2804

Merged
keith merged 22 commits intomasterfrom
adin/align-resource_actions-with-upstream
Nov 12, 2025
Merged

Align actool with upstream#2804
keith merged 22 commits intomasterfrom
adin/align-resource_actions-with-upstream

Conversation

@adincebic adincebic changed the title Align actool action with upstream Align actool with upstream Nov 2, 2025
@adincebic adincebic force-pushed the adin/align-resource_actions-with-upstream branch 2 times, most recently from c701f3f to 998a982 Compare November 12, 2025 01:04
nglevin and others added 14 commits November 11, 2025 20:04
…g analysis time logic to set them as necessary.

PiperOrigin-RevId: 779168158
(cherry picked from commit f83c6e8)
…happy paths of watchOS and macOS icon bundles with tests.

PiperOrigin-RevId: 779317298
(cherry picked from commit a46ce2f)
PiperOrigin-RevId: 779343706
(cherry picked from commit a8fe45c)
…ation of provided icons and launch images, moving the actual validation to actool.bzl.

In the process, we consider more corner cases like forbidding legacy icon assets in platforms that should never use them, if provided via resources from deps and not just top level attributes.

PiperOrigin-RevId: 780974060
(cherry picked from commit 46824ac)
… composer bundles for Xcode 26 beta 2.

PiperOrigin-RevId: 781044757
(cherry picked from commit 7a371a5)
…ts of Xcode 26 beta 3.

This version no longer generates a set of icons for previous versions of iOS/macOS when given an Icon Composer bundle and no icons for an asset catalog.

PiperOrigin-RevId: 781063789
(cherry picked from commit 8721963)
…ppIcons in xcassets if they are targeting a minimum_os_version prior to 26.0 when available.

PiperOrigin-RevId: 781437521
(cherry picked from commit b286806)
… visible warning, but not an error, for Xcode 26 compatibility.

PiperOrigin-RevId: 781523933
(cherry picked from commit 4c37c0c)
Keeping rules flexible to accomodate users on previous betas at the moment, leaving the "actool" warnings and errors alone for the time being.

PiperOrigin-RevId: 786434481
(cherry picked from commit 3906137)
…eta 5.

PiperOrigin-RevId: 793627429
(cherry picked from commit c345148)
…e 26 betas.

PiperOrigin-RevId: 804411520
(cherry picked from commit 34f1048)
…6 beta 5 and later; this was only used by the Xcode 26 betas up through beta 4.

PiperOrigin-RevId: 804575923
(cherry picked from commit e21ad32)
PiperOrigin-RevId: 805044535
(cherry picked from commit 239c0e9)
@adincebic adincebic force-pushed the adin/align-resource_actions-with-upstream branch from 998a982 to d95588a Compare November 12, 2025 01:05
@keith keith marked this pull request as ready for review November 12, 2025 21:26
@keith keith merged commit d240302 into master Nov 12, 2025
11 checks passed
@keith keith deleted the adin/align-resource_actions-with-upstream branch November 12, 2025 21:27
adincebic referenced this pull request Nov 12, 2025
…g analysis time logic to set them as necessary.

PiperOrigin-RevId: 779168158
adincebic referenced this pull request Nov 12, 2025
…happy paths of watchOS and macOS icon bundles with tests.

PiperOrigin-RevId: 779317298
adincebic referenced this pull request Nov 12, 2025
…ation of provided icons and launch images, moving the actual validation to actool.bzl.

In the process, we consider more corner cases like forbidding legacy icon assets in platforms that should never use them, if provided via resources from deps and not just top level attributes.

PiperOrigin-RevId: 780974060
adincebic referenced this pull request Nov 12, 2025
… composer bundles for Xcode 26 beta 2.

PiperOrigin-RevId: 781044757
adincebic referenced this pull request Nov 12, 2025
…ts of Xcode 26 beta 3.

This version no longer generates a set of icons for previous versions of iOS/macOS when given an Icon Composer bundle and no icons for an asset catalog.

PiperOrigin-RevId: 781063789
adincebic referenced this pull request Nov 12, 2025
…ppIcons in xcassets if they are targeting a minimum_os_version prior to 26.0 when available.

PiperOrigin-RevId: 781437521
adincebic referenced this pull request Nov 12, 2025
… visible warning, but not an error, for Xcode 26 compatibility.

PiperOrigin-RevId: 781523933
adincebic referenced this pull request Nov 12, 2025
Keeping rules flexible to accomodate users on previous betas at the moment, leaving the "actool" warnings and errors alone for the time being.

PiperOrigin-RevId: 786434481
adincebic referenced this pull request Nov 12, 2025
adincebic referenced this pull request Nov 12, 2025
adincebic referenced this pull request Nov 12, 2025
…6 beta 5 and later; this was only used by the Xcode 26 betas up through beta 4.

PiperOrigin-RevId: 804575923
adincebic referenced this pull request Nov 12, 2025
adincebic pushed a commit that referenced this pull request Dec 12, 2025
#2804 introduced in issue
where comparisons with `platform_type` are always negative.
`platform_type` is of type `apple_platform_type` and cannot be compared
with `string` directly.
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.

4 participants