8000 Add Permissions to Ultralytics Actions `format.yml` by glenn-jocher · Pull Request #85 · ultralytics/thop · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add Permissions to Ultralytics Actions format.yml #85

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 11, 2025
Merged

Add Permissions to Ultralytics Actions format.yml #85

merged 1 commit into from
May 11, 2025

Conversation

glenn-jocher
Copy link
Member
@glenn-jocher glenn-jocher commented May 10, 2025 •
edited by UltralyticsAssistant
Loading

This pull request significantly enhances the Ultralytics Actions workflow, introducing permissions for security best practices, alongside intelligent issue and PR handling. This includes automatic labeling, spam detection, abuse identification, and off-topic filtering. These updates ensure a more efficient and organized workflow, adhering to the latest Ultralytics standards.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improved the GitHub workflow to enhance code formatting, labeling, and AI-powered automation for PRs and issues. 🚀🤖

📊 Key Changes

  • Updated workflow name and descriptions for clarity.
  • Expanded permissions to allow the workflow to write to code, PRs, and issues.
  • Renamed the job from "format" to "actions" to reflect broader functionality.
  • Enhanced Ultralytics Actions usage:
    • Added support for AI-powered auto-labeling and PR summaries.
    • Improved Python formatting with Ruff and docformatter.
    • Enabled spelling checks with codespell.
    • Added support for Brave API key for better link resolution.

🎯 Purpose & Impact

  • Streamlines and automates code formatting, labeling, and documentation tasks.
  • Improves PR and issue management with AI-driven summaries and labels.
  • Reduces manual effort for maintainers and contributors, ensuring higher code quality and consistency.
  • Makes the contribution process smoother and more user-friendly for everyone.

@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops enhancement New feature or request labels May 10, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @glenn-jocher, thank you for submitting an ultralytics/thop 🚀 PR! To ensure a seamless integration of your work, please review the following checklist:

  • ✅ Define a Purpose: Clearly explain the purpose of your fix or feature in your PR description, and link to any relevant issues. Ensure your commit messages are clear, concise, and adhere to the project's conventions.
  • ✅ Synchronize with Source: Confirm your PR is synchronized with the ultralytics/thop main branch. If it's behind, update it by clicking the 'Update branch' button or by running git pull and git merge main locally.
  • ✅ Ensure CI Checks Pass: Verify all Ultralytics Continuous Integration (CI) checks are passing. If any checks fail, please address the issues.
  • ✅ Update Documentation: Update the relevant documentation for any new or modified features.
  • ✅ Add Tests: If applicable, include or update tests to cover your changes, and confirm that all tests are passing.
  • ✅ Sign the CLA: Please ensure you have signed our Contributor License Agreement if this is your first Ultralytics PR by writing "I have read the CLA Document and I sign the CLA" in a new message.
  • ✅ Minimize Changes: Limit your changes to the minimum necessary for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." — Bruce Lee

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! 🚀

This is an automated response. An Ultralytics engineer will review your PR and assist you soon! 🤖✨

@glenn-jocher glenn-jocher merged commit d768aa8 into main May 11, 2025
3 checks passed
@glenn-jocher glenn-jocher deleted the update-format-workflow-20250511015707 branch May 11, 2025 00:21
@UltralyticsAssistant
Copy link
Member

Exciting to see these improvements merged—thank you, @glenn-jocher! As Henry Ford said, “If everyone is moving forward together, then success takes care of itself.” Your enhancements to our workflows bring that spirit to life, making Ultralytics even more efficient, collaborative, and welcoming for all contributors. We appreciate your dedication to raising the bar for code quality and community experience! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
devops GitHub Devops or MLops enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@glenn-jocher @UltralyticsAssistant
0