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

Merged
merged 2 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

Updated licensing references and added consistency in GitHub workflow files 🛠️✨.

📊 Key Changes

  • Updated licensing comments in GitHub workflow files to maintain a consistent format:
  • No functional code changes were made.

🎯 Purpose & Impact

  • 🔒 Improved License Clarity: Ensures clear visibility of the AGPL-3.0 license in workflow files, aiding compliance and transparency.
  • 🧹 Consistency: Provides uniform documentation headers across files, improving maintainability and professionalism.
  • No direct impact on users or code functionality but streamlines project management.

@UltralyticsAssistant UltralyticsAssistant added dependencies Dependencies and packages documentation Improvements or additions to documentation labels Jan 15, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @pderrenger, thank you for submitting a ultralytics/xview-yolov3 🚀 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 if relevant, link to any associated issues. The more detail, the better!
  • Synchronize with Source: Ensure your PR is synchronized with the ultralytics/xview-yolov3 main branch. If it's out of sync, update it by clicking the 'Update branch' button here on GitHub or by running git pull and git merge main locally.
  • Verify CI Passes: Confirm all Ultralytics Continuous Integration (CI) checks are passing. CI failures may require updates to your code.
  • Update Documentation: If applicable, update the relevant documentation to reflect any new or modified features in your PR.
  • Add or Modify Tests: For any functional changes, include or revise tests to confirm your modifications work as expected. Ensure all tests pass!
  • Sign the CLA: If this is your first contribution to Ultralytics, please sign our Contributor License Agreement (CLA) by commenting "I have read the CLA Document and I sign the CLA".
  • Ensure Minimal Changes: Keep your code changes to the minimum essential for your updates. 🦾 "Simplicity is the soul of efficiency." – Austin Freeman

For additional guidance, refer to the Contributing Guide. If you've encountered a bug, consider providing a minimum reproducible example (MRE) to help us reproduce and debug the issue faster. 🚀

Thank you for contributing to Ultralytics! An Ultralytics engineer will review your PR shortly and provide further feedback. 😃

@pderrenger pderrenger merged commit 59a749e into main Jan 15, 2025
3 checks passed
@pderrenger pderrenger deleted the refactor-20250115173012 branch January 15, 2025 16:33
@UltralyticsAssistant
Copy link
Member

🎉 Your PR has been merged—thank you, @pderrenger, with contributions from @glenn-jocher, for your attention to detail and commitment to excellence! ✨

The updated licensing references and workflow consistency may seem small, but as Aristotle wisely said, "Well begun is half done." These improvements lay a stronger foundation for clarity, professionalism, and future growth within the project.

Your contributions make a difference—keep building greatness, team! 🚀💡

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

Successfully merging this pull request may close these issues.

3 participants