Skip to content

Commit

Permalink
xtask: Fix lint --scope=only-fuzz trying and failing to build_web().
Browse files Browse the repository at this point in the history
  • Loading branch information
kpreid committed Nov 8, 2024
1 parent 73eb6bf commit c50c2e4
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions tools/xtask/src/xtask.rs
Original file line number Diff line number Diff line change
Expand Up @@ -565,6 +565,10 @@ fn build_web(
time_log: &mut Vec<Timing>,
profile: Profile,
) -> Result<(), ActionError> {
// Currently, wasm is considered part of the main workspace scope,
// even though it is actually a separate workspace. This is a bug.
// <https://github.com/kpreid/all-is-cubes/issues/270>
// <https://github.com/kpreid/all-is-cubes/issues/410>
assert!(config.scope.includes_main_workspace());

let wasm_package_dir: &Path = &PROJECT_DIR.join("all-is-cubes-wasm");
Expand Down Expand Up @@ -686,10 +690,10 @@ fn do_for_all_packages(
) -> Result<(), ActionError> {
if config.scope.includes_main_workspace() {
ensure_wasm_tools_installed(config, time_log)?;
}

// Ensure all-is-cubes-server build that might be looking for the web client files will succeed.
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.
Expand Down

0 comments on commit c50c2e4

Please sign in to comment.