Ultralytics Refactor https://ultralytics.com/actions

[glenn-jocher]

This Ultralytics PR refactors code to improve performance and readability. 🔄

Key changes include:

• 🚀 Optimized various functions for faster execution.
• 🧩 Simplified complex logic for better understanding and maintenance.
• 🗑️ Removed redundant code to streamline operations.
• 📚 Improved code structure and organization.

These changes aim to enhance the overall quality and efficiency of the code. 🌟

Learn more about Ultralytics:

• About Us 🌐
• Install the Ultralytics App 📲
• Read Our Blog 📝
• Learn about YOLO 🤖
• Explore Ultralytics HUB 🚀
• Check Out Our Plans 💼
• Join our Team 🌾
• AI in Healthcare 🏥
• AI in Manufacturing 🏭
• Attend YOLO VISION 2024 📅

Summary by Sourcery

CI:

• Update the format workflow to use either the _GITHUB_TOKEN or GITHUB_TOKEN secret.

🛠️ PR Summary

_{Made with ❤️ by Ultralytics Actions}

🌟 Summary

Improved GitHub Actions formatting workflow by adjusting the token handling.

📊 Key Changes

• Updated the format.yml workflow to use a fallback mechanism for GitHub tokens by replacing ${{ secrets._GITHUB_TOKEN }} with ${{ secrets._GITHUB_TOKEN || secrets.GITHUB_TOKEN }}.

🎯 Purpose & Impact

• 🛠 Increased robustness: Ensures the workflow can handle scenarios where _GITHUB_TOKEN is unavailable by falling back to the standard GITHUB_TOKEN.
• 🌟 Improved reliability: This change minimizes disruptions in automated tasks like formatting and labeling, benefiting contributors and maintainers alike.

[sourcery-ai]

Reviewer's Guide by Sourcery

This PR refactors code to improve performance and readability by optimizing various functions, simplifying complex logic, removing redundant code, and improving code structure and organization.

Sequence diagram for GitHub Actions workflow with Ultralytics formatting

sequenceDiagram
    participant GH as GitHub Workflow
    participant UA as Ultralytics Action
    participant Fmt as Formatters

    GH->>UA: Run with token
    Note over UA: Use GITHUB_TOKEN or _GITHUB_TOKEN
    UA->>Fmt: Apply formatting
    Note over Fmt: - Python code & docstrings<br/>- YAML, JSON, Markdown, CSS
    UA->>GH: Apply labels to issues/PRs
    UA->>GH: Submit formatted changes

Loading

File-Level Changes

Change	Details	Files
Updated the GitHub Actions workflow to use either the _GITHUB_TOKEN or GITHUB_TOKEN secret.	Changed the token used in the Ultralytics action from secrets._GITHUB_TOKEN to `secrets._GITHUB_TOKEN

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[UltralyticsAssistant]

👋 Hello @glenn-jocher, thank you for submitting this ultralytics/google-images-download 🚀 PR! To facilitate a smooth review and integration process, please take note of the following checklist:

• ✅ Define a Purpose: Ensure your PR description clearly conveys the purpose of these changes and links to any relevant issues. Clear and concise commit messages are also appreciated and should follow the repository's conventions.
• ✅ Synchronize with Source: Make sure your PR is up-to-date with the main branch of ultralytics/google-images-download. If not, update it by clicking the 'Update branch' button or running git pull and git merge main locally.
• ✅ Verify CI Success: Check that all Continuous Integration (CI) pipelines pass successfully. Address any errors to ensure everything runs smoothly.
• ✅ Update Documentation: If applicable, please update the documentation to reflect any new functionality or changes made.
• ✅ Include Tests: If your PR modifies or adds features, include or update tests to verify the changes, and confirm all tests pass.
• ✅ Sign the CLA: If this is your first contribution to Ultralytics, ensure you’ve signed the Contributor License Agreement. You can do so by commenting, "I have read the CLA Document and I sign the CLA" below.
• ✅ Reduce Scope: Keep your changes succinct and focused to the relevant topic. Remember, "The wise coder eliminates the unnecessary." 😉

If your PR is related to a bug fix and does not already include a Minimum Reproducible Example (MRE), kindly provide one to help us understand and validate the issue.

For further guidance, you can refer to our Contributing Guide. If you have any concerns or questions, feel free to leave a comment.

Thank you for your contributions to Ultralytics! An engineer will review your PR soon. 🙌🚀

[sourcery-ai]

Hey @glenn-jocher - I've reviewed your changes - here's some feedback:

Overall Comments:

• The PR description does not match the actual changes. Please update the description to focus on the specific change being made (adding a token fallback) rather than generic refactoring claims and marketing links.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[UltralyticsAssistant]

🎉 Hey @glenn-jocher, your PR has been merged—thank you for enhancing the robustness of our GitHub Actions workflow! 🛠️✨ Your thoughtful improvements ensure smoother collaboration for everyone involved.

As Marcus Aurelius once said, “What stands in the way becomes the way.” Your fix turned potential obstacles into opportunities for more resilient processes. 🌟 Every detail you refine helps build a stronger, more reliable foundation for all contributors. Onward and upward! 🚀