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 #22

Merged
merged 1 commit 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

Updated format.yml workflow header for better readability and consistency. ✨

📊 Key Changes

  • Adjusted the comment styling in format.yml file to enhance clarity.

🎯 Purpose & Impact

  • 🛠️ Improved Readability: The updated comment structure ensures clearer communication about the file's purpose and license.
  • 📄 Consistency: Aligns the formatting with Ultralytics' standards, fostering uniformity across workflows.

No functional changes were made—this is strictly a documentation improvement! 🚀

@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops documentation Improvements or additions to documentation labels Jan 15, 2025
@UltralyticsAssistant
Copy link
Member

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

  • Define a Purpose: Clearly explain the purpose of your update or improvement in your PR description. Include any relevant issues your PR addresses. Detailed commit messages are highly encouraged!
  • Synchronize with Source: Please ensure your branch is up-to-date with the latest ultralytics/kinect main branch. If it's outdated, merge it with the latest changes using git pull and git merge main or click 'Update branch'.
  • Ensure CI Checks Pass: Verify that all Continuous Integration (CI) checks are passing. Check out our CI documentation for additional troubleshooting or context if needed.
  • Review Documentation Updates: If relevant, update the corresponding documentation to reflect any changes you've made.
  • Add or Update Tests: Ensure your PR includes proper test cases to verify new or changed functionality, and that existing tests remain unaffected.
  • Sign the CLA: If this is your first contribution, sign the Contributor License Agreement (CLA) by commenting "I have read the CLA Document and I sign the CLA" below.
  • Minimize Scope: Keep changes concise and focused solely on your intended improvement or bug fix. As Bruce Lee said, "It is not the daily increase but daily decrease. Hack away the unessential."

Please also note that this is an automated response 🤖, and an Ultralytics engineer will review your PR shortly. Thank you for your attention to detail and for helping in making the ultralytics/kinect repository even better! 🙌

For additional information on how to contribute, check out our Contributing Guide. If you have any questions, drop a comment here anytime.

🚀 Thank you for being a part of the Ultralytics community!

@pderrenger pderrenger merged commit 39c48c4 into main Jan 15, 2025
1 check passed
@pderrenger pderrenger deleted the refactor-20250115173012 branch January 15, 2025 16:32
@UltralyticsAssistant
Copy link
Member

🎉 PR Merged! Huge thanks to @pderrenger for leading this improvement and @glenn-jocher for your valuable contributions! Your updates to format.yml may seem subtle, but they exemplify the attention to detail that elevates great projects to extraordinary ones. ✨

As Leonardo da Vinci once said, "Simplicity is the ultimate sophistication." Your work to fine-tune readability and consistency will certainly resonate far beyond this single workflow. 🚀 Thank you for upholding the spirit of excellence at Ultralytics! ❤️

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

Successfully merging this pull request may close these issues.

3 participants