Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update links.yml to 900s delay #127

Merged
merged 1 commit into from
Nov 16, 2024
Merged

Update links.yml to 900s delay #127

merged 1 commit into from
Nov 16, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 16, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

πŸ”— Adjusted link-checking workflow to improve efficiency.

πŸ“Š Key Changes

  • Increased the retry delay from 300 seconds (5 minutes) to 900 seconds (15 minutes) in the GitHub workflow for checking links.

🎯 Purpose & Impact

  • Benefit: Reduces frequent retry attempts, allowing transient issues more time to resolve naturally.
  • Impact: More efficient use of resources and potentially fewer false failure alerts, leading to a smoother user experience in managing link checks.

@UltralyticsAssistant UltralyticsAssistant added the devops GitHub Devops or MLops label Nov 16, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting a docs repository πŸš€ PR! To ensure a smooth review and integration process, please consider the checklist below:

  • βœ… Define a Purpose: Clearly articulate the purpose of your change in this PR's description, and link to any relevant issues. Make sure your commit messages are clear and in line with project guidelines.
  • βœ… Synchronize with Source: Verify that your PR is aligned with the latest version of the main branch for ultralytics/docs. If needed, update your branch using the 'Update branch' button or by running git pull followed by git merge main locally.
  • βœ… Ensure CI Checks Pass: Check that all Ultralytics Continuous Integration (CI) tests pass successfully. If any tests fail, please address the issues.
  • βœ… Update Documentation: If your change involves new or updated features, ensure that relevant documentation is updated accordingly.
  • βœ… Add Tests: For any new functionalities or modifications, include or update tests and verify that all tests pass.
  • βœ… Sign the CLA: If this is your first contribution to Ultralytics, ensure you have signed our Contributor License Agreement by adding "I have read the CLA Document and I sign the CLA" in a new comment.
  • βœ… Minimize Changes: Aim to keep your changes minimal and focused. Reflect on Bruce Lee's wisdom: "It is not daily increase but daily decrease, hack away the unessential."

This is an automated response, but rest assured that an Ultralytics engineer will review your PR shortly. If you have any questions or need further assistance, feel free to leave a comment. Thank you for your contribution and helping improve the Ultralytics infrastructure! πŸš€βœ¨

@glenn-jocher glenn-jocher merged commit 2fcdb63 into main Nov 16, 2024
2 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch November 16, 2024 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops GitHub Devops or MLops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants