mdlinkcheck: only look for markdown links in markdown files #19938
Conversation
It finds debug outputs in source code otherwise. Output the whitelist "warnings" to stderr to better allow us to count URLs with `./mdlinkcheck | wc -l`.
|
augment review |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
scripts/mdlinkcheck
Outdated
| return; | ||
|
|
||
| # is it a markdown extension? | ||
| my $md = ($f =~ /\.md$/); |
There was a problem hiding this comment.
Consider making the extension check case-insensitive (e.g., /\.md$/i) so files like README.MD are also scanned. This avoids missing markdown links in repos that use uppercase extensions.
🤖 Was this useful? React with 👍 or 👎
There was a problem hiding this comment.
Pull request overview
This PR improves the mdlinkcheck script to prevent false positives when scanning for broken links. The script now restricts markdown-style link detection ([text](url)) to only markdown files, avoiding the detection of debug output patterns in source code. Additionally, whitelist warnings are redirected to stderr to enable clean URL counting via ./mdlinkcheck | wc -l.
- Adds markdown file detection to only search for markdown-style links in
.mdfiles - Redirects whitelist warnings to stderr for cleaner stdout output
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1 participant
It finds debug outputs in source code otherwise.
Output the whitelist "warnings" to stderr to better allow us to count URLs with
./mdlinkcheck --dry-run | wc -l.