Skip to content

Lychee link checker: cache results and accept 429s (likely a valid page but rate limited)#2631

Merged
IsaacG merged 1 commit intoexercism:mainfrom
IsaacG:lychee
Mar 8, 2026
Merged

Lychee link checker: cache results and accept 429s (likely a valid page but rate limited)#2631
IsaacG merged 1 commit intoexercism:mainfrom
IsaacG:lychee

Conversation

@IsaacG
Copy link
Copy Markdown
Member

@IsaacG IsaacG commented Mar 7, 2026

  • Add --cache to avoid fetching the same page multiple times.
  • Accept HTTP/429 results (rate limiting) to make this test less flaky when rate limited; a future run ought to pick up broken links here.

@IsaacG IsaacG requested a review from a team as a code owner March 7, 2026 18:46
@IsaacG IsaacG added x:action/improve Improve existing functionality/content x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows) x:size/tiny Tiny amount of work x:rep/tiny Tiny amount of reputation labels Mar 7, 2026
@IsaacG
Copy link
Copy Markdown
Member Author

IsaacG commented Mar 7, 2026

+cc @ErikSchierboom for write permissions

@IsaacG IsaacG merged commit 42a67d2 into exercism:main Mar 8, 2026
7 of 8 checks passed
@IsaacG IsaacG deleted the lychee branch March 8, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:action/improve Improve existing functionality/content x:rep/tiny Tiny amount of reputation x:size/tiny Tiny amount of work x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants