Avatar for the jeertmans user
jeertmans
languagetool-rust
BlogDocsChangelog

refactor: further separate CLI logic from the API related functionality (see #117)

#124Merged
Comparing
Rolv-Apneseth:refactor-v3
(
c7d342c
) with
v3
(
a7247e4
)
CodSpeed Performance Gauge
-6%
Improvements
0
Regressions
0
Untouched
6
New
0
Dropped
0
Ignored
0

Benchmarks

Passed

large
benches/benchmarks/check_texts.rs::checks::bench_split::split::large
CodSpeed Performance Gauge
-4%
126.6 ms
132.5 ms
medium
benches/benchmarks/check_texts.rs::checks::bench_basic::basic::medium
CodSpeed Performance Gauge
-6%
109 ms
115.5 ms
medium
benches/benchmarks/check_texts.rs::checks::bench_split::split::medium
CodSpeed Performance Gauge
-6%
112.8 ms
119.6 ms
small
benches/benchmarks/check_texts.rs::checks::bench_split::split::small
CodSpeed Performance Gauge
-6%
109.3 ms
116.1 ms
large
benches/benchmarks/check_texts.rs::checks::bench_basic::basic::large
CodSpeed Performance Gauge
-6%
109.1 ms
115.9 ms
small
benches/benchmarks/check_texts.rs::checks::bench_basic::basic::small
CodSpeed Performance Gauge
-6%
108.5 ms
115.4 ms

Commits

Click on a commit to change the comparison range
Base
v3
a7247e4
0%
chore: bump minimum rust version
a6ea313
3 months ago
by Rolv-Apneseth
0%
fix: misc CI issues
15d1454
3 months ago
by Rolv-Apneseth
0%
feat: use `enum_dispatch` to avoid needing to manually call `cmd.execute` for each variant of `Command`
673ff5d
3 months ago
by Rolv-Apneseth
0%
refactor: avoid cloning request input string
2575c87
3 months ago
by Rolv-Apneseth
0%
refactor: use `Cow<'static, str>` instead of `String` for `check::Request.text`
b863f36
3 months ago
by Rolv-Apneseth
-6%
refactor: use `Cow<'source, str>` when not compiling with the `cli` feature Required cloning structs and methods from `api::check` to separate out the `clap` functionality, as `clap` wouldn't support the lifetime without it being `'static`
a1b258d
2 months ago
by Rolv-Apneseth
0%
fix: correct length addition and use `Cow::to_mut`
befbc52
2 months ago
by Rolv-Apneseth
0%
fix: satisfy `clippy` pre-commit hook
c7d342c
2 months ago
by Rolv-Apneseth
Home Terms PrivacyDocs