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

Standardize license headers in TOML/YAML files #244

Merged
merged 3 commits into from
Jan 15, 2025

Conversation

pderrenger
Copy link
Member

@pderrenger pderrenger commented Jan 15, 2025

This PR updates all YAML file headers to use a standardized license format. 🔄

Changes:

  • 📝 Standardized header: # Ultralytics 🚀 AGPL-3.0 License - https://ultralytics.com/license
  • 🧹 Ensures consistent spacing after headers
  • 🔍 Applies to all YAML files except those in venv

Learn more about Ultralytics licensing 📚

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Added an updated Ultralytics license banner across all repository configuration files and workflows.

📊 Key Changes

  • Updated all headers in GitHub workflows, issue templates, and configuration files to include the new licensing format: "Ultralytics 🚀 AGPL-3.0 License - https://ultralytics.com/license".
  • Standardized licensing reference for clarity and branding consistency.

🎯 Purpose & Impact

  • 🛡️ Legal Clarity: Ensures clear attribution of AGPL-3.0 licensing across all repository files, helping developers understand usage terms more easily.
  • 🌐 Branding Improvement: Aligns all repository content with the Ultralytics brand identity, providing a polished and cohesive user experience.
  • 📢 Transparency: Reinforces Ultralytics' commitment to open-source licensing practices, fostering trust in the community.

No functional changes were introduced, so this update is purely administrative. 🚀

@UltralyticsAssistant UltralyticsAssistant added the documentation Improvements or additions to documentation label Jan 15, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @pderrenger, thank you for submitting a PR to ultralytics/hub-sdk! 🚀 This is an automated response to help guide your contribution through the review process. An Ultralytics engineer will review your pull request shortly. Meanwhile, please review the checklist below to ensure a smooth integration:

  • Define a Purpose: In your PR description, clearly state the purpose of your changes and link any relevant issues, if applicable. For example, explain how your standardized license headers benefit the project.
  • Synchronize with Source: Confirm your branch is up to date with the main branch. If it’s outdated, merge the latest changes from main by clicking the 'Update branch' button or by running git pull and git merge main locally.
  • Ensure CI Checks Pass: Verify that all Continuous Integration (CI) checks are successful. Address any errors or warnings to ensure compliance.
  • Update Documentation: If your changes modify functionality or processes, please update the corresponding documentation.
  • Add Tests: Add or update tests as necessary to validate your changes. Run the test suite locally to confirm all tests pass.
  • Sign the CLA: If this is your first contribution to Ultralytics, review and sign our Contributor License Agreement (CLA). To do this, comment on this PR with "I have read the CLA Document and I sign the CLA."
  • Minimize Changes: Double-check that your proposed changes are as focused and minimal as possible to accomplish your stated purpose. “Simplicity is the ultimate sophistication.”

🔍 In reference to the changes you've made:

  • Your update to standardize license headers is a great move toward consistency and compliance. To avoid unnecessary merge conflicts, ensure no unrelated changes or modifications are included in this PR.
  • For the licensing text changes like # Ultralytics 🚀 AGPL-3.0 License - https://ultralytics.com/license, verify that all files subject to this update are covered and correctly formatted.

For more details about contributing, please consult our Contributing Guide. 💡 If you have any questions, feel free to ask in this PR, and someone will assist you.

Thank you for your contribution to Ultralytics! 🚀

@glenn-jocher glenn-jocher merged commit 909d5aa into main Jan 15, 2025
5 checks passed
@glenn-jocher glenn-jocher deleted the refactor-20250115173012 branch January 15, 2025 16:49
@UltralyticsAssistant
Copy link
Member

✨ The PR is merged—thank you, @pderrenger and @glenn-jocher, for your outstanding contributions! ✨ Your work updating the Ultralytics license banner ensures transparency, branding consistency, and legal clarity for the community. 🚀

As Marcus Aurelius once said, "What we do now echoes in eternity." Your efforts today may seem administrative, yet they lay the foundation for trust and excellence, resonating far beyond this moment. Keep inspiring, one commit at a time! 🌟

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants