fix: remove deprecated --exclude-mail flag from lychee config [FaaFyfxR9WAQrL7FcAgEHJvztd8cVMxvjHRS55rw1nwH] - #8080
Conversation
…xR9WAQrL7FcAgEHJvztd8cVMxvjHRS55rw1nwH]
|
Welcome to RustChain! Thanks for your first pull request. Before we review, please make sure:
Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150) A maintainer will review your PR soon. Thanks for contributing! |
|
Verified and this is correct. Current lychee has no That means the removal is behaviour-preserving as well as necessary: nothing starts getting checked that was not being checked before. Single line, single purpose, no test impact. Looks good to me. One scheduling note: #8082 also deletes this same line as part of an unrelated docs change, so the two conflict in |
Per maintainer review: the --exclude rules hide broken links instead of fixing them, and the --exclude-mail removal collides with PR Scottcjn#8080. This PR is now the single-line docs correction only.
Fixes #8077
The --exclude-mail flag was removed in a newer version of lychee, causing the link checker to fail with "unexpected argument '--exclude-mail' found" on every run since July 22. Mail links are skipped by default now, so the flag is no longer needed.
This is a one-line fix: simply remove the --exclude-mail argument from the lychee args.