Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

add better warning when prebuilding in non-interactive mode #4456

Merged
merged 1 commit into from
Aug 1, 2022

Conversation

wkozyra95
Copy link
Contributor

Why

currently, if someone has a managed project with an ios directory, or is writing eas secrets to ios directory e.g.

echo "$IOS_GOOGLE_SERVICES_BASE64" | base64 -d > ios/GoogleService-Info.plist

then only warning they are getting that ios directory is deleted is log

- Clearing ios
✔ Cleared ios code

How

print working when rebuilding in non-interactive mode

Test Plan

@wkozyra95 wkozyra95 requested review from EvanBacon and dsokal July 5, 2022 14:09
@@ -104,6 +104,9 @@ export async function promptToClearMalformedNativeProjectsAsync(
initial: true,
}))
) {
if (isNonInteractive()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

It's unclear if any changes landed here will be published, I recommend opening here https://github.com/expo/expo/blob/main/packages/%40expo/cli/src/prebuild/clearNativeFolder.ts

@wkozyra95 wkozyra95 merged commit bf44f0c into main Aug 1, 2022
@wkozyra95 wkozyra95 deleted the @wkozyra95/add-warning-when-clearing branch August 1, 2022 10:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants