Skip to content

Create cla.yml#88

Merged
glenn-jocher merged 1 commit into
mainfrom
glenn-jocher-patch-1
Aug 4, 2025
Merged

Create cla.yml#88
glenn-jocher merged 1 commit into
mainfrom
glenn-jocher-patch-1

Conversation

@glenn-jocher
Copy link
Copy Markdown
Member

@glenn-jocher glenn-jocher commented Aug 4, 2025

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Added an automated Contributor License Agreement (CLA) check for pull requests to ensure all contributors sign the required agreement before their code is merged. 🤝

📊 Key Changes

  • Introduced a new GitHub Actions workflow (cla.yml) to manage CLA signing.
  • Automatically prompts PR authors to sign the Ultralytics CLA.
  • Verifies CLA signatures before allowing PRs to be merged.
  • Supports custom comments and status updates for CLA signing.

🎯 Purpose & Impact

  • Ensures all code contributions are legally compliant and properly authorized. 🛡️
  • Streamlines the contribution process by automating CLA checks, reducing manual effort for maintainers.
  • Provides clear feedback to contributors about their CLA status, making the process more transparent and user-friendly. 🚦

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops enhancement New feature or request labels Aug 4, 2025
@UltralyticsAssistant
Copy link
Copy Markdown
Member

👋 Hello @glenn-jocher, thank you for submitting an ultralytics/thop 🚀 PR! This is an automated response to help you ensure a smooth integration of your contribution. An Ultralytics engineer will also review and assist you soon.

  • Define a Purpose: Please make sure your PR description clearly explains the purpose of your changes, and link to any relevant issues. Ensure your commit messages are clear and follow the project's conventions.
  • Synchronize with Source: Confirm your branch is up to date with the ultralytics/thop main branch. If necessary, update via the 'Update branch' button or by running git pull and git merge main locally.
  • Ensure CI Checks Pass: Check that all Ultralytics Continuous Integration (CI) checks are passing. If any fail, please resolve them.
  • Update Documentation: Update the documentation to reflect any new or modified features.
  • Add Tests: If applicable, include or update tests to cover your changes, and verify that all tests pass.
  • Sign the CLA: Please sign our Contributor License Agreement (CLA) if this is your first Ultralytics PR by commenting "I have read the CLA Document and I sign the CLA" in this PR.
  • Minimize Changes: Limit changes to only what's necessary for your addition or fix. "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 see our Contributing Guide. If you have any questions, feel free to leave a comment. Thank you for contributing to Ultralytics! 🚀✨

@glenn-jocher glenn-jocher merged commit 45fc717 into main Aug 4, 2025
4 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch August 4, 2025 07:36
@UltralyticsAssistant
Copy link
Copy Markdown
Member

🎉 Fantastic work, @glenn-jocher! This merge is a big step forward for the Ultralytics community. As Helen Keller said, “Alone we can do so little; together we can do so much.” Your addition of automated CLA checks empowers collaboration while ensuring compliance and transparency for everyone. Thank you for helping make contributing to Ultralytics smoother and more welcoming! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops GitHub Devops or MLops enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants