Skip to content

Conversation

@MikeMcC399
Copy link
Collaborator

Situation

Executing npm run check:markdown results in errors checking the ./factory/README.md document:

  ERROR: 3 dead links found!
  [✖] https://www.npmjs.com/package/yarn → Status: 403
  [✖] https://www.npmjs.com/package/cypress → Status: 403
  [✖] https://www.npmjs.com/package/geckodriver → Status: 403

These are HTTP 403 Forbidden errors, resulting from the npm registry refusing to honor non-browser requests.

Change

  • Add ^https://www.npmjs.com to ignorePatterns in markdownLinkConfig.json

  • Remove the existing "pattern": "^#S*". This is not necessary.

Verification

npm ci
npm run check:markdown

Confirm that there are no link issues reported.

Comment

Due to a bug in markdown-link-check the return code is always set to success. Once this is fixed, the check:markdown script could be added to CI to report on link issues.

@cypress-app-bot
Copy link

@MikeMcC399 MikeMcC399 added the bug label Sep 21, 2025
@MikeMcC399 MikeMcC399 self-assigned this Sep 21, 2025
@MikeMcC399 MikeMcC399 marked this pull request as ready for review September 21, 2025 09:51
@jennifer-shehane jennifer-shehane merged commit 888b094 into cypress-io:master Sep 22, 2025
51 checks passed
@MikeMcC399 MikeMcC399 deleted the fix/npm-registry-link-checks branch September 22, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants