Skip to content

chore(lints): introduce typos-cli config in Cargo.toml and handle typos#2390

Open
mkatychev wants to merge 8 commits intonickel-lang:masterfrom
mkatychev:chore/typos-cli-lint
Open

chore(lints): introduce typos-cli config in Cargo.toml and handle typos#2390
mkatychev wants to merge 8 commits intonickel-lang:masterfrom
mkatychev:chore/typos-cli-lint

Conversation

@mkatychev
Copy link
Contributor

related PR: topiary/topiary#832

@mkatychev mkatychev marked this pull request as draft October 21, 2025 21:44
@mkatychev
Copy link
Contributor Author

Not sure where this line/column diff is coming from 🤔 https://github.com/tweag/nickel/actions/runs/18698880001/job/53323091230?pr=2390#step:5:1848

@jneem
Copy link
Member

jneem commented Oct 22, 2025

It may be best to redo this after #2302 is finished -- otherwise there will be a lot of conflicts

@mkatychev
Copy link
Contributor Author

Sounds good, all the manual steps have been documented so I can just rerun all of it.

@mkatychev mkatychev force-pushed the chore/typos-cli-lint branch from 45acfbc to ac49c90 Compare March 2, 2026 17:07
@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

🐰 Bencher Report

Branchchore/typos-cli-lint
Testbedubuntu-latest
Click to view all benchmark results
BenchmarkLatencymicroseconds (µs)
diagnostics-benches/inputs/goto-perf.ncl📈 view plot
🚷 view threshold
10,296.00 µs
diagnostics-benches/inputs/large-record-tree.ncl📈 view plot
🚷 view threshold
183,890.00 µs
diagnostics-benches/inputs/redis-replication-controller.ncl📈 view plot
🚷 view threshold
330.00 µs
diagnostics-benches/inputs/small-record-tree.ncl📈 view plot
🚷 view threshold
454.89 µs
fibonacci 10📈 view plot
🚷 view threshold
212.56 µs
foldl arrays 50📈 view plot
🚷 view threshold
373.40 µs
foldl arrays 500📈 view plot
🚷 view threshold
2,710.50 µs
foldr strings 50📈 view plot
🚷 view threshold
2,906.80 µs
foldr strings 500📈 view plot
🚷 view threshold
27,621.00 µs
generate normal 250📈 view plot
🚷 view threshold
13,367.00 µs
generate normal 50📈 view plot
🚷 view threshold
937.40 µs
generate normal unchecked 1000📈 view plot
🚷 view threshold
1,670.20 µs
generate normal unchecked 200📈 view plot
🚷 view threshold
337.54 µs
init-diagnostics-benches/inputs/goto-perf.ncl📈 view plot
🚷 view threshold
53,455.00 µs
init-diagnostics-benches/inputs/large-record-tree.ncl📈 view plot
🚷 view threshold
202,370.00 µs
init-diagnostics-benches/inputs/redis-replication-controller.ncl📈 view plot
🚷 view threshold
48,726.00 µs
init-diagnostics-benches/inputs/small-record-tree.ncl📈 view plot
🚷 view threshold
48,953.00 µs
pidigits 100📈 view plot
🚷 view threshold
1,868.00 µs
pipe normal 20📈 view plot
🚷 view threshold
590.65 µs
pipe normal 200📈 view plot
🚷 view threshold
4,382.80 µs
product 30📈 view plot
🚷 view threshold
250.48 µs
requests-benches/inputs/goto-perf.ncl-000📈 view plot
🚷 view threshold
3,058.30 µs
requests-benches/inputs/large-record-tree.ncl-000📈 view plot
🚷 view threshold
600,790.00 µs
requests-benches/inputs/large-record-tree.ncl-001📈 view plot
🚷 view threshold
105.13 µs
scalar 10📈 view plot
🚷 view threshold
499.72 µs
sum 30📈 view plot
🚷 view threshold
257.02 µs
🐰 View full continuous benchmarking report in Bencher

sd 'seach' 'search' ./**/*.rs
sd 'seach' 'search' ...(fd -e rs)
sd 'dyamic' 'dynamic' ./**/*.md
sd 'dyamic' 'dynamic' ...(fd)
sd 'wich' 'which' ...(fd)
sd 'contining' 'containing' ./**/*.rs
sd 'contining' 'containing' ...(fd)
sd 'contine' 'continue' ...(fd)
sd 'evalution' 'evaluation' ...(fd)
typos --no-check-filenames
sd 'overrided' 'overridden' ...(fd)
^sd 'doens' 'doesn' ./**/*.rs
^sd 'shrinked' 'shrunk' ./**/*.rs
sd 'represention' 'representation' ...(fd)
^sd 'confing' 'config' ./**/*.md
^sd 'chaning' 'changing' ./**/*.rs
@mkatychev mkatychev force-pushed the chore/typos-cli-lint branch from ac49c90 to f6bee86 Compare March 2, 2026 17:48
@mkatychev mkatychev marked this pull request as ready for review March 2, 2026 17:49
@mkatychev
Copy link
Contributor Author

@jneem this should be ready for review now

Copy link
Member

@jneem jneem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

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