v3 - Branch - jeertmans/languagetool-rust

chore(lib): fully refactor the library for v3

#117Merged

Comparing

(

880f1a9

) with

main

(

7cadfe5

)

-51%

Improvements: 0

Regressions: 6

Untouched: 0

New: 0

Dropped: 0

Ignored: 0

Benchmarks

Passed

largeRegression

benches/benchmarks/check_texts.rs::checks::bench_split::split::large

-46%

101 ms

186.2 ms

mediumRegression

benches/benchmarks/check_texts.rs::checks::bench_split::split::medium

-49%

87.5 ms

172.9 ms

smallRegression

benches/benchmarks/check_texts.rs::checks::bench_split::split::small

-51%

83.8 ms

169.6 ms

largeRegression

benches/benchmarks/check_texts.rs::checks::bench_basic::basic::large

-51%

83.5 ms

169.3 ms

mediumRegression

benches/benchmarks/check_texts.rs::checks::bench_basic::basic::medium

-51%

83.3 ms

169 ms

smallRegression

benches/benchmarks/check_texts.rs::checks::bench_basic::basic::small

-51%

83.1 ms

168.9 ms

Commits

Click on a commit to change the comparison range

Base

main

7cadfe5

-6%

refactor: further separate CLI logic from the API related functionality (see #117) (#124) * refactor: use `std::ops::Not::not` instead of custom `is_false` function * refactor: further separate API logic from CLI, and create submodules for each subcommand * fix: fmt * fix: nightly `rustfmt` warning: the `version` option is deprecated. Use `style_edition` instead. * fix: fmt * refactor: `ProcessCommand` -> `process::Command` * fix: remove unused imports * refactor: make link more readable for CLI * fix: `clippy` and `doc` warnings * chore: bump minimum rust version * fix: misc CI issues * feat: use `enum_dispatch` to avoid needing to manually call `cmd.execute` for each variant of `Command` * refactor: avoid cloning request input string * refactor: use `Cow<'static, str>` instead of `String` for `check::Request.text` * chore(lib): adding Cow * 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` * fix: remove unnecessary clone of `split_pattern` * refactor: use `Cow<'source, str>` for text referenced by `ResponseWithContext` * Update CI.yml * Update README.md Co-authored-by: Jérome Eertmans <jeertmans@icloud.com> * Update benches/benchmarks/check_texts.rs Co-authored-by: Jérome Eertmans <jeertmans@icloud.com> * Update benches/benchmarks/check_texts.rs Co-authored-by: Jérome Eertmans <jeertmans@icloud.com> * Update src/api/server.rs Co-authored-by: Jérome Eertmans <jeertmans@icloud.com> * Update tests/match_positions.rs Co-authored-by: Jérome Eertmans <jeertmans@icloud.com> * fix: formatting * fix: remove unused imports * fix: remove static lifetime from `with_text` and `with_data` * fix: correct length addition and use `Cow::to_mut` * fix: satisfy `clippy` pre-commit hook --------- Co-authored-by: Jérome Eertmans <jeertmans@icloud.com>

a067522

7 months ago

by Rolv-Apneseth

refactor: remove unnecessary code duplication from `cli/check`

a90bd15

7 months ago

by Rolv-Apneseth

ci: include newer versions of `languagetool` in tests (#126) * ci: include newer versions of `languagetool` in tests * tests: debug errors with server testing * ci: `--no-capture` for testing library code, for easier debugging failing CI tests

ccb494a

7 months ago

by Rolv-Apneseth

refactor(lib): consolidate `reqwest` errors (#125) * refactor: consolidate `Error::ResponseDecode` and `Error::ResponseEncode` into just `Error::Reqwest` * tests: `Error::Reqwest` * tests: check that an invalid request is returned when the input is too long when checking either text or annotated data

1665a9d

6 months ago

by Rolv-Apneseth

+31%

feat: add support for HTML, Markdown and Typst files (#127) * feat: add parsers for HTML, markdown and typst files * feat: in the CLI, use different parsers based on file type * fix: formatting * fix: bump MSRV * fix: features ordering * refactor: update markdown file extensions * refactor: use data annotations for `typst` files

82bd45b

4 months ago

by Rolv-Apneseth

-61%

chore(lib): cleanup and logging

027f1ed

4 months ago

by jeertmans

fix(tests): increase size of request

6cae2f4

4 months ago

by jeertmans

fix(tests): set env to limit max text length

cf9462b

4 months ago

by jeertmans

-15%

Merge remote-tracking branch 'origin/main' into v3

d14c0b9

3 months ago

by jeertmans

fix(tests): enforce language

7e9e6cc

3 months ago

by jeertmans

ea24862

3 months ago

by jeertmans

Merge branch 'main' into v3

6ccefd7

22 days ago

by Rolv-Apneseth

fix(ci): run the LanguageTool service for the coverage CI to avoid failing tests (#137) * fix(ci): run the LanguageTool service for the coverage CI so that tests don't fail * fix: formatting issue in Cargo.toml

b07d7c1

22 days ago

by Rolv-Apneseth

fix(ci): version for languagetool in coverage CI (#138)

abee1db

22 days ago

by Rolv-Apneseth

Merge branch 'main' into v3

044edf1

9 days ago

by jeertmans

ci(dependabot): bump actions/checkout from 2 to 4 (#148) Bumps [actions/checkout](https://github.com/actions/checkout) from 2 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v2...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '4' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

44e7208

9 days ago

by dependabot[bot]

ci(dependabot): bump pre-commit-ci/lite-action from 1.0.2 to 1.1.0 (#145) --- updated-dependencies: - dependency-name: pre-commit-ci/lite-action dependency-version: 1.1.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

650fcc6

9 days ago

by dependabot[bot]

ci(dependabot): bump tarides/changelog-check-action from 2 to 3 (#146) --- updated-dependencies: - dependency-name: tarides/changelog-check-action dependency-version: '3' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

7cffe92

9 days ago

by dependabot[bot]

ci(dependabot): bump moonrepo/setup-rust from 0 to 1 (#147) Bumps [moonrepo/setup-rust](https://github.com/moonrepo/setup-rust) from 0 to 1. - [Release notes](https://github.com/moonrepo/setup-rust/releases) - [Changelog](https://github.com/moonrepo/setup-rust/blob/master/CHANGELOG.md) - [Commits](https://github.com/moonrepo/setup-rust/compare/v0...v1) --- updated-dependencies: - dependency-name: moonrepo/setup-rust dependency-version: '1' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

42727f9

9 days ago

by dependabot[bot]

ci(dependabot): bump codecov/codecov-action from 2 to 5 (#149) Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 2 to 5. - [Release notes](https://github.com/codecov/codecov-action/releases) - [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md) - [Commits](https://github.com/codecov/codecov-action/compare/v2...v5) --- updated-dependencies: - dependency-name: codecov/codecov-action dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

b1f960a

9 days ago

by dependabot[bot]

fix(ci): LanguageTool env vars for coverage CI (#139) * fix(ci): LanguageTool env vars for coverage CI * fixup!: bump Java max mem for LanguageTool * fixup!: bump checkout action version * chore(ci): run ping for debug * fixup!: try coverage without the `cargo-tarpaulin` container * fixup!: install `cargo-tarpaulin` using `taiki-e/install-action` * fixup!: remove debug pings * fix: accept output of the latest LanguageTool container * fix: solve lint issue with the `yaml` pre-commit hook for `dependabot.yml` --------- Co-authored-by: Jérome Eertmans <jeertmans@icloud.com>

0c7eccd

8 days ago

by Rolv-Apneseth

preparing for merge

4191a93

7 days ago

by jeertmans

chore(docs): add changelog entries and remove useless features

8bc0d26

7 days ago

by jeertmans

fixes

92e6431

7 days ago

by jeertmans

chore(tests): update markdown snapshot

1d460ea

6 days ago

by Rolv-Apneseth

chore: add `docker-compose` file to simplify running a local API for tests (#153)

cc1bcaa

2 days ago

by Rolv-Apneseth

Merge remote-tracking branch 'origin/v3' into v3

ecd702f

2 days ago

by jeertmans

chore: lint code and update pre-commit config

0044017

1 day ago

by jeertmans

update docs

3a9b20d

1 day ago

by jeertmans

chore(ci): disable a bunch of LT rules

2172f71

1 day ago

by jeertmans

chore(docs): include note about snapshot testing in `CONTRIBUTING.md`

eca34d4

1 day ago

by Rolv-Apneseth

fix(ci): avoid running the coverage CI more than once on a PR (both push and pull)

da2d5f0

1 day ago

by Rolv-Apneseth

fix(ci): run changelog action on the latest Ubuntu version (retirement error due to old version)

880f1a9

1 day ago

by Rolv-Apneseth

Home Terms Privacy Docs