Skip to content

ci: check markdown links with maintainer_tools action - #385

Merged
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:markdown-link-check
Oct 10, 2026
Merged

blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:markdown-link-check

Conversation

@blink1073

Copy link
Copy Markdown
Contributor

Adds a markdown-link-check job using calysto/maintainer_tools/actions/markdown-link-check@v1 and wires it into tests_check. Removes the now-redundant tcort/markdown-link-check pre-commit hook and the .markdown_link_config.json file, since the action's defaults cover the links we check.

@blink1073
blink1073 enabled auto-merge (squash) October 10, 2026 21:07
@greptile-apps

greptile-apps Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Low impact] The PR appears safe to merge; no actionable issue was established.

Summary

Moves Markdown link checks from a manual pre-commit hook into CI.

  • GitHub Actions checks Markdown links as part of the required checks.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Checkout repository] --> B[Check Markdown links]
  B --> C[tests_check]
  D[Existing test jobs] --> C
Loading

Reviews (1) · Last reviewed commit: "ci: use markdown-link-check action in te..." · Reviewed by Greptile

@codecov

codecov Bot commented Oct 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.03%. Comparing base (b4a80d0) to head (0c83b75).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #385   +/-   ##
=======================================
  Coverage   93.03%   93.03%           
=======================================
  Files           6        6           
  Lines         445      445           
=======================================
  Hits          414      414           
  Misses         31       31           
Flag Coverage Δ
linux 92.80% <ø> (ø)
windows 93.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@blink1073
blink1073 merged commit d391438 into Calysto:main Oct 10, 2026
52 of 55 checks passed
@blink1073
blink1073 deleted the markdown-link-check branch October 10, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant