- Notifications
You must be signed in to change notification settings - Fork5.7k
Stabilize Linkcheck Test#4693
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
PR Overview
The PR aims to stabilize the linkcheck tests by adjusting the configuration for known network issues and by capturing the linkcheck output for easier debugging.
- In docs/source/conf.py, a comment and regex escape were added to bypass issues with the LGPL license link.
- In the GitHub workflow, a new step was added to upload linkcheck output as an artifact.
Reviewed Changes
| File | Description |
|---|---|
| docs/source/conf.py | Added regex escape for LGPL license link and accompanying comment. |
| .github/workflows/docs-linkcheck.yml | Added a workflow step to upload the linkcheck output artifact. |
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
Uh oh!
There was an error while loading.Please reload this page.
77c2593 intomasterUh oh!
There was an error while loading.Please reload this page.
Also uploads the output as artifact so that it's easier to locally check for the failed entries