Skip to content

Replace markdown-link-check with lychee - #177

Merged
mauran merged 3 commits into
mainfrom
copilot/replace-markdown-link-check
Oct 2, 2026
Merged

mauran merged 3 commits into
mainfrom
copilot/replace-markdown-link-check

Conversation

Copilot AI commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

Replace the Markdown link checker with lychee while preserving pull-request and scheduled checks.

  • Workflow: Use lycheeverse/lychee-action@v2 with existing retry, timeout, and accepted-status behavior; restrict token permissions to contents: read.
  • Exclusions: Move previously suppressed hosts to .lycheeignore and remove obsolete checker configs and README markers.
uses: lycheeverse/lychee-action@v2
with:
  args: --verbose --no-progress --timeout 60 --max-retries 3 --retry-wait-time 5 --max-concurrency 100 --accept 200..=299,403,429,500,503 './**/*.md'

Copilot AI and others added 3 commits October 2, 2026 09:41
Co-authored-by: mauran <282398+mauran@users.noreply.github.com>
Co-authored-by: mauran <282398+mauran@users.noreply.github.com>
Co-authored-by: mauran <282398+mauran@users.noreply.github.com>
@mauran
mauran marked this pull request as ready for review October 2, 2026 09:44
Copilot AI balanced review requested due to automatic review settings October 2, 2026 09:45
@mauran
mauran merged commit 774bc59 into main Oct 2, 2026
@mauran
mauran deleted the copilot/replace-markdown-link-check branch October 2, 2026 09:45

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot review overview

馃煝 Approval recommended

The workflow and exclusion migration match the stated scope, with no unresolved blocking findings.

Review effort: Balanced
Findings: None

What changed in this PR

Replaces the repository鈥檚 Markdown link checker with lychee while retaining pull-request and scheduled checks.

Changes:

  • Configures lychee with retry, timeout, concurrency, and accepted-status settings.
  • Restricts workflow permissions to contents: read.
  • Moves exclusions into .lycheeignore and removes obsolete configuration and README markers.
File Description
README.md Removes legacy link-check suppression comments.
.lycheeignore Adds exclusions for previously suppressed hosts.
.github/鈥媤orkflows/鈥媗ychee.yml Switches to lychee and limits token permissions.
.github/鈥媘lc_config.json Deletes obsolete checker configuration.
.github/鈥媘arkdown-link-check-config.json Deletes the replaced checker configuration.

馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

3 participants