-
Notifications
You must be signed in to change notification settings - Fork 85
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
[eas-cli] mark default
image tag and old images as deprecated and add new image tags to eas.schema.json
#2298
Conversation
default
image tag as deprecateddefault
image tag as deprecated and add new image tags to eas.schema.json
Size Change: -43 B (0%) Total Size: 51.4 MB
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2298 +/- ##
==========================================
- Coverage 53.67% 53.65% -0.01%
==========================================
Files 525 525
Lines 19160 19167 +7
Branches 4043 4047 +4
==========================================
Hits 10282 10282
- Misses 8152 8158 +6
- Partials 726 727 +1 ☔ View full report in Codecov by Sentry. |
/changelog-entry chore Deprecate the |
/changelog-entry new-feature Add |
/changelog-entry chore Deprecate Android images with JDK 8 and NDK |
1aa370d
to
fe8fd8a
Compare
default
image tag as deprecated and add new image tags to eas.schema.json
default
image tag and old images as deprecated and add new image tags to eas.schema.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering if we should also include some simple suggestion to follow for developers in the eas-cli Android warning. Not knowing what I know about images I wouldn't want to need to pick a replacement for a deprecated Android image. Which? Why?
"This image is deprecated, please use one of the more recent images instead." | ||
"This image tag is deprecated, please use other image tags or images directly instead.", | ||
"This image tag is deprecated, please use other image tags or images directly instead.", | ||
"This image is using Xcode in version lower then 15. Apple's new requirement states that starting on April 29th, 2024, apps uploaded to App Store Connect must be built with Xcode 15 for iOS 17, iPadOS 17, tvOS 17, or watchOS 10. Start using Xcode 15 image today by setting `macos-ventura-13.6-xcode-15.2` or `latest` as your image value. This image will be removed from EAS Build on May 29th, 2024. Learn more: expo.dev/changelog/2024/03-29-eas-build-upcoming-ios-images-updates.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the URL have https? I think I saw vscode linkifying links in descriptions?
@sjchmiela You are right, I will remove the Android image warnings for now because I will be able to give people more helpful instructions on what to do next once https://github.com/expo/turtle-v2/pull/1744 lands and we have more images with newer NDKs better suited for SDK 49 and 50. Will open a separate PR to add Android images warnings 👍. |
fe8fd8a
to
5c9dbd3
Compare
✅ Thank you for adding the changelog entry! |
Why
The current
default
image tag is confusing. It's not even the true default. Changing it breaks people's builds and so on...Let's deprecate it.
Add a new
auto
tag that is an explicit way of opting into the true default behavior (auto-resolving image when it is not specified ineas.json
).Add
sdk-XY
tagsDeprecate images with Xcode < 15 because of https://expo.dev/changelog/2024/03-29-eas-build-upcoming-ios-images-updates. Advise people on how to switch to Xcode 15 image.
Deprecate old Android images.
More context: https://exponent-internal.slack.com/archives/C9PRD479V/p1710257552292179
expo/expo#27852
https://github.com/expo/turtle-v2/pull/1736
How
Mark the
default
image tag as deprecatedAdd a new
auto
tag that is an explicit way of opting into the true default behavior (auto-resolving image when it is not specified ineas.json
).Add
sdk-XY
tagsDeprecate images with Xcode < 15 because of https://expo.dev/changelog/2024/03-29-eas-build-upcoming-ios-images-updates. Advise people on how to switch to Xcode 15 image.
Deprecate old Android images with JDK 8 (used in auto image resolving logic for RN < 0.68.0) and NDK r19c (used for SDK < 46).
Test Plan
Test manually