Skip to content

add/lychee-docker-local-make-call - #2465

Open
Micha Hobert (Isengo1989) wants to merge 6 commits into
mainfrom
add/local-lychee
Open

add/lychee-docker-local-make-call#2465
Micha Hobert (Isengo1989) wants to merge 6 commits into
mainfrom
add/local-lychee

Conversation

@Isengo1989

Copy link
Copy Markdown
Contributor

Summary

Added local Markdown link checking via Lychee in Docker:

make linkcheck — runs lycheeverse/lychee with the same HTTPS accept/exclude rules as CI
DIR= — optional folder scope, e.g. make linkcheck DIR=guides/plugins
Root-relative portal links ignored — --root-dir /var/empty so paths like /frontends/ don’t fail (they belong to the developer portal, not this repo)

Usage: make linkcheck or make linkcheck DIR=path/to/folder.

Related links

Checklist

  • I reviewed affected links, code samples, and cross-references, including PageRef references where relevant.
  • I added or updated redirects in .gitbook.yaml if pages were moved, renamed, or deleted.
  • I updated .wordlist.txt (and sorted it) if spellcheck flags new legitimate terms.
  • Any required dependent changes in downstream modules have already been merged and published.
  • This pull request is ready for review.

Notes

Copilot AI lite review requested due to automatic review settings August 21, 2026 09:08
@shopware-dev-docs-connector

shopware-dev-docs-connector Bot commented Aug 21, 2026

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Completed with success.
Repository: shopware/docs
Commit: 6454d5d
Preview: https://developer-documentation-68d2kxh49-shopware-frontends.vercel.app
Workflow run: #5011

@github-actions

Copy link
Copy Markdown
Contributor

📢 Developer Announcement Recommendation

Recommendation: ❌ No Announcement Recommended

No developer announcement signals were detected.

This PR does not appear to introduce developer-facing changes that require a community announcement.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds Dockerized local Lychee Markdown link checking with optional directory scoping and CI-aligned URL rules.

Changes:

  • Defines Lychee Docker configuration and arguments.
  • Adds make linkcheck with optional DIR support.
Suppressed comments (1)

Makefile:37

  • The new contributor-facing command is not documented in the README's existing External link check section, which currently only points to the scheduled CI workflow. Please add the make linkcheck and DIR=... usage there so contributors can discover the local check instead of relying on the PR description or make help.
linkcheck: ## Check HTTPS links in Markdown (via Docker / Lychee). Optional: DIR=path/to/folder

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

Comment thread Makefile Outdated
@Isengo1989 Micha Hobert (Isengo1989) added the Improvement PR created to append/modify info in the existing article label Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement PR created to append/modify info in the existing article

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants