Merge pull request #484 from csmoe/valid

validate target argument
master
ashley williams 6 years ago committed by GitHub
commit 4cd77901c3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 8
      src/command/build.rs

@ -88,7 +88,7 @@ pub struct BuildOptions {
pub scope: Option<String>, pub scope: Option<String>,
#[structopt(long = "mode", short = "m", default_value = "normal")] #[structopt(long = "mode", short = "m", default_value = "normal")]
/// Sets steps to be run. [possible values: no-install, normal] /// Sets steps to be run. [possible values: no-install, normal, force]
pub mode: BuildMode, pub mode: BuildMode,
#[structopt(long = "no-typescript")] #[structopt(long = "no-typescript")]
@ -158,6 +158,12 @@ impl Build {
_ => bail!("Can only supply one of the --dev, --release, or --profiling flags"), _ => bail!("Can only supply one of the --dev, --release, or --profiling flags"),
}; };
// `possible_values` in clap isn't supported by `structopt`
let possible_targets = ["browser", "nodejs", "no-modules"];
if !possible_targets.contains(&build_opts.target.as_str()) {
bail!("Supported targets: browser, nodejs, no-modules");
}
Ok(Build { Ok(Build {
crate_path, crate_path,
crate_data, crate_data,

Loading…
Cancel
Save