Skip to content

Add TOML formatting checks#1121

Open
xtqqczze wants to merge 3 commits intoreubeno:mainfrom
xtqqczze:tombi
Open

Add TOML formatting checks#1121
xtqqczze wants to merge 3 commits intoreubeno:mainfrom
xtqqczze:tombi

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

@xtqqczze xtqqczze commented May 3, 2026

No description provided.

Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Performance Benchmark Report

Benchmark name Baseline (μs) Test/PR (μs) Delta (μs) Delta %
clone_shell_object 13.66 μs 13.72 μs 0.06 μs ⚪ Unchanged
eval_arithmetic 0.12 μs 0.12 μs 0.00 μs ⚪ Unchanged
expand_one_string 1.22 μs 1.22 μs 0.00 μs ⚪ Unchanged
for_loop 24.09 μs 25.08 μs 0.99 μs 🟠 +4.10%
full_peg_complex 43.48 μs 43.83 μs 0.35 μs ⚪ Unchanged
full_peg_for_loop 4.78 μs 4.82 μs 0.04 μs ⚪ Unchanged
full_peg_nested_expansions 12.79 μs 12.85 μs 0.06 μs ⚪ Unchanged
full_peg_pipeline 3.25 μs 3.26 μs 0.01 μs ⚪ Unchanged
full_peg_simple 1.36 μs 1.38 μs 0.02 μs ⚪ Unchanged
function_call 2.73 μs 3.02 μs 0.29 μs ⚪ Unchanged
instantiate_shell 43.29 μs 42.62 μs -0.67 μs 🟢 -1.55%
instantiate_shell_with_init_scripts 21237.39 μs 21356.44 μs 119.04 μs ⚪ Unchanged
parse_peg_bash_completion 1621.17 μs 1623.32 μs 2.15 μs ⚪ Unchanged
parse_peg_complex 14.66 μs 15.03 μs 0.37 μs 🟠 +2.54%
parse_peg_for_loop 1.45 μs 1.45 μs 0.00 μs ⚪ Unchanged
parse_peg_pipeline 1.56 μs 1.54 μs -0.02 μs ⚪ Unchanged
parse_peg_simple 0.82 μs 0.84 μs 0.02 μs 🟠 +2.56%
run_echo_builtin_command 13.10 μs 12.98 μs -0.12 μs ⚪ Unchanged
tokenize_sample_script 2.69 μs 2.74 μs 0.05 μs 🟠 +1.90%

Code Coverage Report: Only Changed Files listed

Package Base Coverage New Coverage Difference
Overall Coverage 🟢 75.2% 🟢 75.2% ⚪ 0%

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

Test Summary: bash-completion test suite

Outcome Count Percentage
✅ Pass 1582 75.01
❗️ Error 18 0.85
❌ Fail 155 7.35
⏩ Skip 339 16.07
❎ Expected Fail 13 0.62
✔️ Unexpected Pass 2 0.09
📊 Total 2109 100.00

@xtqqczze xtqqczze marked this pull request as ready for review May 4, 2026 00:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants