Skip to content

docs: replace nonexistent test:all script with npm test - #4054

Open
FirmaSpring wants to merge 1 commit into
markedjs:masterfrom
FirmaSpring:docs-fix-test-all-script
Open

docs: replace nonexistent test:all script with npm test#4054
FirmaSpring wants to merge 1 commit into
markedjs:masterfrom
FirmaSpring:docs-fix-test-all-script

Conversation

@FirmaSpring

Copy link
Copy Markdown

Problem

The contributor checklist in docs/CONTRIBUTING.md tells contributors to run npm run test:all, but no script named test:all exists in package.json. Following the documented step fails:

$ npm run test:all
npm error Missing script: "test:all"
npm error
npm error Did you mean one of these?
npm error   npm install # Install a package
npm error   npm run test:cjs # run the "test:cjs" package script

Solution

Replace npm run test:all with npm test in the checklist. The test script in package.json runs the full suite (build:reset, build:docs, test:specs, test:unit, test:umd, test:cjs, test:types, test:lint), which is the complete check a contributor needs before submitting.

Verification

  • Confirmed test:all does not exist in package.json (rg '"test' package.json lists all test scripts; none is test:all).
  • Confirmed npm run test:all fails with "Missing script".
  • Confirmed npm test is the full-suite script defined in package.json.

The contributor checklist references npm run test:all, but no such script exists in package.json

Co-authored-by: FirmamentalSpring <287222957+FirmaSpring@users.noreply.github.com>
@vercel

vercel Bot commented Aug 17, 2026

Copy link
Copy Markdown

@FirmaSpring is attempting to deploy a commit to the MarkedJS Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant