Exclude build script by default

[taiki-e]

Closes #159

[taiki-e]

bors r+

[bors]

Build succeeded:

• ci

[stouset]

Does this actually remove the build_script_build module from coverage results? By my (potentially-inaccurate) reading it looks like it just adds a flag that lets you opt in or out, but the logic for actually excluding those results hasn't changed. So it looks like if you opt out you just end up with the same behavior as what happened previously, no?

[taiki-e]

the logic for actually excluding those results hasn't changed

No. Changes to src/main.rs will change to ignore artifacts related to build scripts if --include-build-script flag is not passed.

if cx.cov.include_build_script {
    let dir = p.file_name().unwrap().to_string_lossy();
    if !cx.build_script_re.is_match(&dir) {
        return false;
    }
} else {
    // `false` means "ignore this path"
    return false;
}

If all corresponding artifacts are ignored, the file (and the module) will not be displayed in the report.

[taiki-e]

Published in 0.4.0.

[stouset]

I must have misunderstood the contents of the patch then. Thank you for the quick turnaround!

[stouset]

Confirmed that this works as expected now. Thanks again!