fluencelabs/wasm-bindgen
1
0
Fork 0
mirror of https://github.com/fluencelabs/wasm-bindgen synced 2025-04-03 02:41:06 +00:00
Code Issues Projects Releases Wiki Activity

Format web-sys bindings by default

Browse Source 
This commit switches to executing `rustfmt` by default on
`web-sys`-generated bindings. This improves situations like "view
source" in Rustdoc as well as the IDE interactive debugging experience.

This was initially disabled by default because `rustfmt` took so long to
execute, but nowadays `web-sys` is by default much smaller so there's
much less need to avoid running `rustfmt` in fear of it taking too
long.

Closes #1457
This commit is contained in:
Alex Crichton 2019-04-15 08:31:54 -07:00
parent 744b031ad4
commit fd4f0b6508
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
crates/web-sys/build.rs
View File

@ -113,13 +113,13 @@ fn try_main() -> Result<(), failure::Error> {
// run rustfmt on the generated file - really handy for debugging // run rustfmt on the generated file - really handy for debugging
println!("cargo:rerun-if-env-changed=WEBIDL_RUSTFMT_BINDINGS"); println!("cargo:rerun-if-env-changed=WEBIDL_RUSTFMT_BINDINGS");
if env::var("WEBIDL_RUSTFMT_BINDINGS").is_ok() { if env::var("WEBIDL_RUSTFMT_BINDINGS").ok() != Some("0".to_string()) {
let status = Command::new("rustfmt") let status = Command::new("rustfmt")
.arg(&out_file_path) .arg(&out_file_path)
.status() .status()
.context("running rustfmt")?; .context("running rustfmt")?;
if !status.success() { if !status.success() {
bail!("rustfmt failed: {}", status) println!("cargo:warning=rustfmt failed: {}", status)
} }
} }