From 5694bb9fdca06b4dd5713c62e86c7995563c1cbb Mon Sep 17 00:00:00 2001 From: Jon Samp Date: Tue, 3 Jan 2023 23:37:49 -0500 Subject: [PATCH] [eas-cli] Makes eas.json configuration to only run on update:configure (#1598) * Makes eas.json configuration to only run on update:configure * update CHANGELOG.md --- CHANGELOG.md | 1 + packages/eas-cli/src/commands/update/configure.ts | 7 ++++++- packages/eas-cli/src/update/configure.ts | 4 +--- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index db35680fd7..26a3c4191a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -13,6 +13,7 @@ This is the log of notable changes to EAS CLI and related packages. ### ๐Ÿ› Bug fixes +- Makes eas.json configuration to only run on `update:configure`. ([#1598](https://github.com/expo/eas-cli/pull/1598) by [@jonsamp](https://github.com/jonsamp)) - Fix issue with invisible build info in some terminals in the `eas build:run` and `eas build:resign` commands. ([#1602](https://github.com/expo/eas-cli/pull/1602) by [@szdziedzic](https://github.com/szdziedzic)) ### ๐Ÿงน Chores diff --git a/packages/eas-cli/src/commands/update/configure.ts b/packages/eas-cli/src/commands/update/configure.ts index 8f075e41a0..826629dc1a 100644 --- a/packages/eas-cli/src/commands/update/configure.ts +++ b/packages/eas-cli/src/commands/update/configure.ts @@ -5,7 +5,10 @@ import EasCommand from '../../commandUtils/EasCommand'; import { EASNonInteractiveFlag } from '../../commandUtils/flags'; import Log from '../../log'; import { RequestedPlatform } from '../../platform'; -import { ensureEASUpdateIsConfiguredAsync } from '../../update/configure'; +import { + ensureEASUpdateIsConfiguredAsync, + ensureEASUpdateIsConfiguredInEasJsonAsync, +} from '../../update/configure'; export default class UpdateConfigure extends EasCommand { static override description = 'configure the project to support EAS Update'; @@ -46,6 +49,8 @@ export default class UpdateConfigure extends EasCommand { platform, }); + await ensureEASUpdateIsConfiguredInEasJsonAsync(projectDir); + Log.addNewLineIfNone(); Log.log(`๐ŸŽ‰ Your app is configured with EAS Update!`); Log.newLine(); diff --git a/packages/eas-cli/src/update/configure.ts b/packages/eas-cli/src/update/configure.ts index ee7f6b7cee..441e291c20 100644 --- a/packages/eas-cli/src/update/configure.ts +++ b/packages/eas-cli/src/update/configure.ts @@ -258,7 +258,7 @@ async function ensureEASUpdateIsConfiguredNativelyAsync( * Make sure EAS Build profiles are configured to work with EAS Update by adding channels to build profiles. */ -async function ensureEASUpdateIsConfiguredInEasJsonAsync(projectDir: string): Promise { +export async function ensureEASUpdateIsConfiguredInEasJsonAsync(projectDir: string): Promise { const easJsonPath = EasJsonAccessor.formatEasJsonPath(projectDir); if (!(await fs.pathExists(easJsonPath))) { @@ -363,8 +363,6 @@ export async function ensureEASUpdateIsConfiguredAsync( workflows, }); - await ensureEASUpdateIsConfiguredInEasJsonAsync(projectDir); - if (projectChanged || !hasExpoUpdates) { await ensureEASUpdateIsConfiguredNativelyAsync(graphqlClient, { exp: expWithUpdates,