Skip to content

fix: avoid highlighting hyphenated shell arguments as flags - #214

Open
xHeaven wants to merge 2 commits into
mainfrom
fix/bash-hyphenated-arguments
Open

fix: avoid highlighting hyphenated shell arguments as flags#214
xHeaven wants to merge 2 commits into
mainfrom
fix/bash-hyphenated-arguments

Conversation

@xHeaven

@xHeaven xHeaven commented Aug 12, 2026

Copy link
Copy Markdown
Member

Fixes -server in demo-server being highlighted as a flag.

@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown

Benchmark Results

Comparison of fix/bash-hyphenated-arguments against main (7eaf2f4030cc218320672902608b6226bb13a39a).

Open to see the benchmark results
benchmark subject set revs its mem_peak mode rstdev
HighlighterBench benchParse apache 100 5 2.066mb 0.00% 69.343346μs -0.75% ±0.71% +14.87%
HighlighterBench benchParse bash 100 5 2.098mb -0.01% 195.332524μs -2.09% ±0.51% -34.09%
HighlighterBench benchParse bbcode 100 5 2.054mb 0.00% 71.363738μs -1.06% ±0.92% +18.47%
HighlighterBench benchParse blade 100 5 2.670mb 0.00% 947.712661μs +0.02% ±0.42% -49.21%
HighlighterBench benchParse css 100 5 2.084mb 0.00% 135.642818μs +0.16% ±0.56% +112.53%
HighlighterBench benchParse diff 100 5 2.049mb 0.00% 25.755127μs -0.18% ±1.66% +25.92%
HighlighterBench benchParse dockerfile 100 5 2.066mb 0.00% 84.183914μs -1.31% ±1.03% +33.46%
HighlighterBench benchParse dotenv 100 5 2.069mb 0.00% 74.838924μs +0.32% ±0.83% +186.36%
HighlighterBench benchParse gdscript 100 5 2.164mb 0.00% 317.572975μs +0.46% ±1.16% +36.40%
HighlighterBench benchParse graphql 100 5 2.085mb 0.00% 145.095793μs -0.41% ±0.33% -66.80%
HighlighterBench benchParse html 100 5 2.088mb 0.00% 111.192329μs -0.24% ±0.34% -6.33%
HighlighterBench benchParse http 100 5 2.058mb 0.00% 22.794834μs +1.10% ±1.40% +155.39%
HighlighterBench benchParse ini 100 5 2.073mb 0.00% 84.013562μs +0.50% ±0.45% -32.12%
HighlighterBench benchParse javascript 100 5 2.152mb 0.00% 258.034932μs -0.02% ±0.46% +3.12%
HighlighterBench benchParse json 100 5 2.127mb 0.00% 230.108395μs -0.33% ±0.32% -18.83%
HighlighterBench benchParse markdown 100 5 2.075mb 0.00% 72.490509μs -0.15% ±1.17% +169.92%
HighlighterBench benchParse nginx 100 5 2.139mb 0.00% 257.182094μs -0.04% ±0.33% -22.23%
HighlighterBench benchParse php 100 5 2.430mb 0.00% 720.952779μs -0.66% ±0.88% +166.37%
HighlighterBench benchParse python 100 5 2.328mb 0.00% 405.204110μs -0.04% ±1.45% +265.74%
HighlighterBench benchParse scss 100 5 2.100mb 0.00% 174.650078μs +0.44% ±0.61% -44.83%
HighlighterBench benchParse sql 100 5 2.099mb 0.00% 149.295068μs -1.20% ±1.00% +21.78%
HighlighterBench benchParse svelte 100 5 2.301mb 0.00% 980.454853μs -0.48% ±0.76% +141.06%
HighlighterBench benchParse terminal 100 5 2.095mb -0.05% 179.742505μs -3.89% ±0.65% +17.63%
HighlighterBench benchParse terraform 100 5 2.099mb 0.00% 166.071018μs +0.73% ±0.57% +1.30%
HighlighterBench benchParse typescript 100 5 2.525mb 0.00% 232.390861μs +0.69% ±0.70% +64.50%
HighlighterBench benchParse twig 100 5 19.206mb 0.00% 1,344.160724μs -0.80% ±0.60% -4.41%
HighlighterBench benchParse vue 100 5 2.382mb 0.00% 367.698748μs +0.34% ±0.42% -48.89%
HighlighterBench benchParse xml 100 5 2.102mb 0.00% 175.012290μs +0.25% ±0.39% -43.12%
HighlighterBench benchParse yaml 100 5 2.160mb 0.00% 313.569609μs -0.07% ±0.43% +0.43%

Generated by phpbench against commit ee2cb3c

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 31644444539

Coverage remained the same at 94.745%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 1 of 1 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 2055
Covered Lines: 1947
Line Coverage: 94.74%
Coverage Strength: 58.92 hits per line

💛 - Coveralls

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