Skip to content

Ignore peerj.com/preprints/1233 in link checker#3681

Merged
tomthorogood merged 1 commit into
mainfrom
jk-ignore-peerj-link
May 29, 2026
Merged

Ignore peerj.com/preprints/1233 in link checker#3681
tomthorogood merged 1 commit into
mainfrom
jk-ignore-peerj-link

Conversation

@kenyonj
Copy link
Copy Markdown
Contributor

@kenyonj kenyonj commented May 29, 2026

PeerJ returns 403 to GitHub Actions runners (likely Cloudflare bot detection), even though the URL resolves fine in a browser. This has been failing CI on recent PRs, e.g. https://github.com/github/opensource.guide/actions/runs/26646297579.

The link is cited in _articles/building-community.md (and all translations) supporting the claim that most projects only have one or two maintainers. The content is still valid, so adding it to the existing url_ignores list matches the pattern already used for other sites that block automated requests.

PeerJ returns 403 to GitHub Actions runners (likely Cloudflare bot
detection), even though the URL resolves fine in a browser. The link is
cited in building-community.md (and all translations) and the content is
still valid, so adding it to the existing url_ignores list matches the
pattern used for other sites that block automated requests.
Copilot AI review requested due to automatic review settings May 29, 2026 15:57
@kenyonj kenyonj requested a review from a team as a code owner May 29, 2026 15:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds the PeerJ preprint URL to the HTMLProofer ignore list because PeerJ returns 403 to GitHub Actions runners (likely Cloudflare bot detection), causing CI failures even though the link is valid.

Changes:

  • Add https://peerj.com/preprints/1233/ to the exact-URL url_ignores array in script/html-proofer.
Show a summary per file
File Description
script/html-proofer Adds the PeerJ preprint URL to the link checker's ignore list.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@tomthorogood tomthorogood added this pull request to the merge queue May 29, 2026
Merged via the queue into main with commit 9433e42 May 29, 2026
6 checks passed
@tomthorogood tomthorogood deleted the jk-ignore-peerj-link branch May 29, 2026 16:04
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.

3 participants