reubeno / brush

bash/POSIX-compatible shell implemented in Rust
MIT License
27 stars 4 forks source link

feat: add a panic handler via human-panic #191

Closed reubeno closed 1 month ago

reubeno commented 1 month ago

The handler will encourage filing a GitHub issue.

github-actions[bot] commented 1 month ago

Test Results

  2 files    8 suites   51s :stopwatch: 423 tests 423 :white_check_mark: 0 :zzz: 0 :x: 846 runs  846 :white_check_mark: 0 :zzz: 0 :x:

Results for commit b0762dc7.

github-actions[bot] commented 1 month ago

Performance Benchmark Report

Benchmark name Baseline (μs) Test/PR (μs) Delta (μs) Delta %
expand_one_string 3.36 μs 3.35 μs -0.01 μs ⚪ Unchanged
instantiate_shell 60.47 μs 59.60 μs -0.87 μs ⚪ Unchanged
instantiate_shell_with_init_scripts 25505.28 μs 25455.45 μs -49.83 μs ⚪ Unchanged
parse_bash_completion 2734.14 μs 2727.13 μs -7.01 μs ⚪ Unchanged
parse_sample_script 4.11 μs 4.08 μs -0.04 μs 🟢 -0.92%
run_echo_builtin_command 90.21 μs 88.50 μs -1.71 μs ⚪ Unchanged
run_one_builtin_command 108.90 μs 109.40 μs 0.50 μs ⚪ Unchanged
run_one_external_command 1883.27 μs 1994.19 μs 110.92 μs 🟠 +5.89%
run_one_external_command_directly 959.04 μs 954.85 μs -4.19 μs ⚪ Unchanged

Code Coverage Report: Only Changed Files listed

Package Base Coverage New Coverage Difference
brush-core/src/jobs.rs 🔴 42.42% 🔴 37.23% 🔴 -5.19%
brush-core/src/shell.rs 🟢 76.28% 🟢 75.98% 🔴 -0.3%
brush-shell/src/main.rs 🟢 93.01% 🟢 90.26% 🔴 -2.75%
Overall Coverage 🟢 72.19% 🟢 72.07% 🔴 -0.12%

Minimum allowed coverage is 70%, this run produced 72.07%