remove/command-and-add-scripts - #2490
Conversation
Developer Docs healthcheckStatus: Completed with |
📢 Developer Announcement RecommendationRecommendation: ❌ No Announcement Recommended No developer announcement signals were detected.
|
There was a problem hiding this comment.
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
pnpmscripts 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-localMake 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.
| spellcheck: ## Runs the spellcheck tool (via Docker) | ||
| docker run --rm -u ${user} -v "$(shell pwd):/docs" -w /docs -e INPUT_IGNORE=${ignored} ${image} \ | ||
| --config /docs/markdown-style-config.yml /docs | ||
|
|
Summary
Remove command from MAKEFILE, but keep it for some period. Added scripts for package.json to replace those
Related links
Checklist
PageRefreferences where relevant..gitbook.yamlif pages were moved, renamed, or deleted..wordlist.txt(and sorted it) if spellcheck flags new legitimate terms.Notes