Skip to content

fix: add missing closing backtick in nose2 description#2951

Open
sprmn24 wants to merge 2 commits intovinta:masterfrom
sprmn24:fix/readme-backtick-typo
Open

fix: add missing closing backtick in nose2 description#2951
sprmn24 wants to merge 2 commits intovinta:masterfrom
sprmn24:fix/readme-backtick-typo

Conversation

@sprmn24
Copy link

@sprmn24 sprmn24 commented Mar 8, 2026

Summary

This PR fixes a missing closing backtick in the nose2 description in README.md.

The description previously rendered incorrectly in Markdown because the backtick before unittest2 was not closed.

Changes

  • Added missing closing backtick in README.md (line 1037)
  • Added GitHub Actions workflow to automatically check links in README.md
  • Added lychee configuration for link validation

Impact

  • Fixes Markdown rendering issue
  • Adds automated link checking to prevent broken links in future PRs

AutoPR Agent added 2 commits March 8, 2026 21:35
The nose2 entry under Testing Frameworks was missing a closing
backtick after `unittest2`, causing downstream markdown to render
as unintended inline code.

Line 1037: based on `unittest2.  ->  based on `unittest2`.
Add automated link checking using lychee (Rust-based, async) to detect
broken URLs in README.md.

Workflow triggers:
  - On PRs that modify README.md
  - On push to master (README.md changes)
  - Weekly scheduled scan (Sunday 06:00 UTC)
  - Manual dispatch

Features:
  - lychee-action v2 with 708+ link validation
  - Persistent cache across runs (3-day max age)
  - Rate limiting (max 10 concurrent, 3 retries, 5s backoff)
  - Custom user-agent to avoid bot detection
  - GITHUB_TOKEN for elevated API rate limits
  - .lycheeignore for known-unreachable domains
  - lychee.toml for accept-429 and timeout tolerance

New files:
  - .github/workflows/link-check.yml
  - lychee.toml
  - .lycheeignore
@vinta
Copy link
Owner

vinta commented Mar 9, 2026

@sprmn24 Please one change per PR.

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.

2 participants