From dd6665a7421ce18b4061d318ab3ad179898b54f7 Mon Sep 17 00:00:00 2001 From: Kevin Reid Date: Thu, 7 Nov 2024 22:11:37 -0800 Subject: [PATCH] xtask: Remove overeager skipping web build. This is now apparently causing a CI failure; I think because I moved the output into `target/` and the cache action is leaving empty directories present. --- tools/xtask/src/xtask.rs | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/tools/xtask/src/xtask.rs b/tools/xtask/src/xtask.rs index d3a25eb97..71b656448 100644 --- a/tools/xtask/src/xtask.rs +++ b/tools/xtask/src/xtask.rs @@ -688,13 +688,8 @@ fn do_for_all_packages( ensure_wasm_tools_installed(config, time_log)?; } - // Ensure all-is-cubes-server build that might be looking for the files will succeed. - // Note that this is only an “exists” check not a “up-to-date” check, on the assumption - // that running server tests will not depend on the specific file contents. - // TODO: That's a fragile assumption. - if config.scope.includes_main_workspace() && !static_web_app_out_dir(Profile::Dev).exists() { - build_web(config, time_log, Profile::Dev)?; - } + // Ensure all-is-cubes-server build that might be looking for the web client files will succeed. + build_web(config, time_log, Profile::Dev)?; // Test everything we can with default features and target. // But if we're linting, then the below --all-targets run will handle that.