Skip to content

remove/command-and-add-scripts - #2490

Merged
Micha Hobert (Isengo1989) merged 3 commits into
mainfrom
remove/local-spellcheck
Sep 2, 2026
Merged

remove/command-and-add-scripts#2490
Micha Hobert (Isengo1989) merged 3 commits into
mainfrom
remove/local-spellcheck

Conversation

@Isengo1989

@Isengo1989 Micha Hobert (Isengo1989) commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove command from MAKEFILE, but keep it for some period. Added scripts for package.json to replace those

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 31, 2026 14:54
@shopware-dev-docs-connector

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

Copy link
Copy Markdown

Developer Docs healthcheck

Status: Completed with success.
Repository: shopware/docs
Commit: 7eea4c2
Preview: https://developer-documentation-5afr2991s-shopware-frontends.vercel.app
Workflow run: #5121

@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

This PR updates contributor tooling by adding package.json scripts intended to replace (or complement) existing Makefile commands, and adjusts ignore patterns used by spellcheck/markdown-lint.

Changes:

  • Add pnpm scripts for link checking, spelling, and markdown auto-fix via Docker.
  • Update spellcheck ignore paths to also skip /docs/resources/references/app-reference/*.
  • Remove the spellcheck-local Make target.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
package.json Adds Docker-based scripts (check:links, check:spelling, fix:markdown) to run common quality checks without using Make targets.
Makefile Updates ignore list and removes spellcheck-local target from the Make-based workflow.

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

Comment thread Makefile
Co-authored-by: Isengo1989 <8600299+Isengo1989@users.noreply.github.com>
Co-authored-by: Isengo1989 <8600299+Isengo1989@users.noreply.github.com>
@Isengo1989 Micha Hobert (Isengo1989) added Improvement PR created to append/modify info in the existing article new Label for new content that is published in the changelog and removed new Label for new content that is published in the changelog labels Sep 2, 2026
@Isengo1989
Micha Hobert (Isengo1989) merged commit 2db4ad1 into main Sep 2, 2026
9 checks passed
@Isengo1989
Micha Hobert (Isengo1989) deleted the remove/local-spellcheck branch September 2, 2026 09:38
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.

5 participants