Restore website link recovery window - #359
Conversation
|
👋 Hello @glenn-jocher, thank you for submitting a
For more guidance, please refer to our Contributing Guide. Don't hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀 |
UltralyticsAssistant
left a comment
There was a problem hiding this comment.
🔍 PR Review
Made with ❤️ by Ultralytics Actions
LGTM. The change restores the intended 1,800-second retry delay while preserving the existing three retries and fixed backoff. The inline comment documents the behavior at its owner, and no compatibility or workflow issues are evident.
|
🎉 PR #359 has been merged—thank you, @glenn-jocher!
The restored 30-minute link-check retry delay gives temporarily unavailable sites more time to recover between attempts, while preserving the existing three retries and fixed backoff configuration. |
Restore the intentional 30-minute delay between website link-check attempts so temporarily unavailable sites have time to recover. PR #358 incorrectly shortened this recovery window to 60 seconds while investigating a GitHub-hosted runner disconnect; the available evidence does not implicate the retry delay.
Keep the existing three retries and fixed backoff, and document the recovery window at its owner. Validation:
actionlintandgit diff --checkpass. The completed four-site run in #358 already verified the unchanged scanning and reporting paths; this correction only restores the delay configuration.🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Restored the website link-check retry delay to 1,800 seconds so temporarily unavailable sites have time to recover between attempts.
📊 Key Changes
.github/workflows/links.ymlto use a 30-minuteretry_delay_secondsvalue instead of 60 seconds.🎯 Purpose & Impact