Skip to content

fix(ci): LanguageTool env vars for coverage CI #139

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
May 26, 2025
Merged

fix(ci): LanguageTool env vars for coverage CI #139

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
dd8bbd7
fix(ci): LanguageTool env vars for coverage CI
Rolv-Apneseth May 12, 2025
b0557b4
fixup!: bump Java max mem for LanguageTool
Rolv-Apneseth May 12, 2025
fbfba6f
fixup!: bump checkout action version
Rolv-Apneseth May 12, 2025
5e363d1
Merge branch 'v3' into fix_coverage_ci
jeertmans May 26, 2025
ebad27b
Merge branch 'v3' into fix_coverage_ci
jeertmans May 26, 2025
c296013
chore(ci): run ping for debug
jeertmans May 26, 2025
af0ade9
fixup!: try coverage without the `cargo-tarpaulin` container
Rolv-Apneseth May 26, 2025
0513e40
fixup!: install `cargo-tarpaulin` using `taiki-e/install-action`
Rolv-Apneseth May 26, 2025
49d3ded
fixup!: remove debug pings
Rolv-Apneseth May 26, 2025
1ac239c
fix: accept output of the latest LanguageTool container
Rolv-Apneseth May 26, 2025
97420c2
fix: solve lint issue with the `yaml` pre-commit hook for `dependabot…
Rolv-Apneseth May 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/dependabot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ updates:
labels:
- dependencies
- rust
target-branch: 'v3' # TODO: change me after merging v3
target-branch: v3 # TODO: change me after merging v3
- package-ecosystem: github-actions
directory: /
schedule:
Expand All @@ -19,4 +19,4 @@ updates:
labels:
- ci
- dependencies
target-branch: 'v3' # TODO: change me after merging v3
target-branch: v3 # TODO: change me after merging v3
18 changes: 14 additions & 4 deletions .github/workflows/coverage.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
name: Code Coverage

on: [push]
on: [push, pull_request]
jobs:
test:
name: Coverage
Expand All @@ -12,13 +12,23 @@ jobs:
- 8010:8010
env:
langtool_maxTextLength: 1500
container:
image: xd009642/tarpaulin:develop-nightly
options: --security-opt seccomp=unconfined
Java_Xmx: 2g
env:
LANGUAGETOOL_HOSTNAME: http://localhost
LANGUAGETOOL_PORT: 8010
steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Install nightly toolchain
uses: dtolnay/rust-toolchain@nightly

- name: Cache dependencies
uses: Swatinem/rust-cache@v2

- name: Install cargo-tarpaulin
uses: taiki-e/install-action@cargo-tarpaulin

- name: Generate code coverage
run: |
cargo +nightly tarpaulin --verbose --all-features --workspace --timeout 120 --out Xml
Expand Down
2 changes: 1 addition & 1 deletion tests/snapshots/cli__autodetect_html_file.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
|
4 | <body> <img /> <img /> <img /> <!-- First coment --> <!-- Feel free to change this text...
| ^^^^^^^^^^^^^^^^^^^^ Possible spelling mistake
| -------------------- help: moment, cement, comment, comet, foment, ... (1 not shown)
| -------------------- help: moment, cement, comment, comet, foment, ... (3 not shown)
|
error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
--> [path]
Expand Down
8 changes: 4 additions & 4 deletions tests/snapshots/cli__autodetect_markdown_file.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,14 +42,14 @@ error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
|
5 | ...piling LTRS. #### Default Features - - **cli**: Adds command-line related methods for ...
| ^^^^^^^ Possible spelling mistake
| ------- help: CLI, Clip, CGI, CPI, CSI, ... (1428 not shown)
| ------- help: CLI, Clip, CGI, CPI, CSI, ... (1506 not shown)
|
error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
--> [path]
|
9 | ...ate**, **color**, **multithreaded**. - **native-tls**: Enables TLS functionality provided by ...
| ^^^^^^^^^^^^^^ Possible spelling mistake
| -------------- help: natives, natively, nativists
| -------------- help: natives, natively, nativists, nativeness, naivetes
|
error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
--> [path]
Expand All @@ -63,14 +63,14 @@ error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
|
6 | ...shells. This feature also activates the **cli** feature. Enter ltrs completions --help ...
| ^^^^^^^ Possible spelling mistake
| ------- help: CLI, clip, CGI, CPI, CSI, ... (1461 not shown)
| ------- help: CLI, clip, CGI, CPI, CSI, ... (1543 not shown)
|
error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
--> [path]
|
10 | ...es color outputting in the terminal. If **cli** feature is also enabled, the --color=<W...
| ^^^^^^^ Possible spelling mistake
| ------- help: CLI, clip, CGI, CPI, CSI, ... (1461 not shown)
| ------- help: CLI, clip, CGI, CPI, CSI, ... (1543 not shown)
|
error[MORFOLOGIK_RULE_EN_US]: Possible spelling mistake found.
--> [path]
Expand Down
Loading