fix(ci): link checker 403 Forbidden errors#5839
Conversation
WalkthroughThe lychee configuration file was updated to exclude "linuxhint.com" from link checks. This change ensures that lychee will skip checking this domain, likely due to bot protection or repeated 403 errors. No other configuration or code logic was altered. Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.config/lychee.toml (1)
20-22: Broaden the pattern to cover all linuxhint.com variants.
lycheetreats eachexcludeentry as a regex;"linuxhint.com"will not match URLs such ashttps://www.linuxhint.com/….
Consider using a simple wildcard/regex to ensure every sub-domain and path is skipped, e.g.:- "linuxhint.com", + ".*linuxhint\\.com.*",(Alternatively, add both
linuxhint.comandwww.linuxhint.com.)Also, the explanatory comment on line 20 duplicates the one on line 19. You can remove it or merge the two for brevity.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.config/lychee.toml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)
- GitHub Check: tests-release
- GitHub Check: Build forest binaries on Linux AMD64
- GitHub Check: tests
- GitHub Check: cargo-publish-dry-run
- GitHub Check: Build Ubuntu
- GitHub Check: Build MacOS
- GitHub Check: All lint checks
|
Thanks! |
Summary of changes
https://github.com/ChainSafe/forest/actions/runs/16317643325/job/46087296773?pr=5835#step:3:1467
Changes introduced in this pull request:
Reference issue to close (if applicable)
Closes
Other information and links
Change checklist
Summary by CodeRabbit