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 Python files #241

Merged
merged 2 commits into from
Jan 15, 2025
Merged

Standardize license headers in Python files #241

merged 2 commits into from
Jan 15, 2025

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 14, 2025

๐Ÿ› ๏ธ PR Summary

Made with โค๏ธ by Ultralytics Actions

๐ŸŒŸ Summary

Updated license references in all project files by adding a URL link to the Ultralytics license for enhanced visibility and compliance. ๐ŸŒ

๐Ÿ“Š Key Changes

  • Replaced the static license comment in each file header with a new format:
    "AGPL-3.0 License - https://ultralytics.com/license" ๐Ÿ“œ
  • Updated headers in all relevant source code and test files to consistently include the new license link.

๐ŸŽฏ Purpose & Impact

  • Purpose

    • Ensures users have direct access to the license details by linking to the official webpage.
    • Promotes clarity and transparency regarding licensing terms.
  • Impact

    • No functional changes to the codebase, ensuring backward compatibility. โœ…
    • Simplifies verification of licensing terms for developers and project contributors. ๐Ÿ’ผ

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

๐Ÿ‘‹ Hello @glenn-jocher, thank you for submitting an ultralytics/hub-sdk ๐Ÿš€ PR! To ensure a seamless integration of your work, please review the following checklist:

  • โœ… Define a Purpose: Clearly explain the purpose of your changes within the PR description. If applicable, link to any relevant issues for this enhancement or bug fix. Ensure your commit messages are clear and follow the repository's conventions.
  • โœ… Synchronize with Source: Confirm your PR is up to date with the latest changes in the main branch of ultralytics/hub-sdk. If itโ€™s outdated, please rebase or merge the latest changes by running git pull followed by git rebase or git merge main.
  • โœ… Ensure CI Checks Pass: Make sure that all Continuous Integration (CI) checks associated with this PR are passing. Address any failing checks or tests.
  • โœ… Update Documentation: If your changes affect the repositoryโ€™s functionality, consider updating the documentation to reflect these updates.
  • โœ… Add Tests: Include or revise tests within the repository to ensure thorough coverage of your changes, guaranteeing their reliability moving forward. Verify that all tests pass locally and in CI workflows.
  • โœ… Sign the CLA: If this is your first contribution to Ultralytics, donโ€™t forget to sign our Contributor License Agreement (CLA) by commenting "I have read the CLA Document and I sign the CLA" within this PR.
  • โœ… Minimize Changes: Keep your modifications focused and limited to the purpose of the PR. Aim to โ€œhack away the unessentialโ€ and streamline your updates to ensure efficient review and integration.

Next Steps ๐Ÿš€

  • Please double-check the PR description for clarity and comprehensiveness, ensuring reviewers can easily understand the proposed changes and their purpose.
  • Follow the guidelines mentioned above to address any necessary updates. For more information, review our Contributing Guide.
  • If you encounter any questions or require additional support, feel free to comment here. An Ultralytics engineer will assist you shortly.

Thank you for your contribution and for being part of the Ultralytics community! ๐Ÿš€

@glenn-jocher glenn-jocher merged commit 4fe06ec into main Jan 15, 2025
7 checks passed
@glenn-jocher glenn-jocher deleted the license branch January 15, 2025 12:18
@UltralyticsAssistant
Copy link
Member

๐ŸŽ‰ PR Merged! Thank you, @glenn-jocher, for your thoughtful updates to the license referencesโ€”bridging clarity and accessibility in every file. ๐ŸŒ Your work ensures that our project's values and transparency are unmistakably present for all contributors and users. ๐Ÿ‘

As Marcus Aurelius once said, "What we do now echoes in eternity." Your efforts here may seem small, but they shape a foundation of trust and integrity that will ripple through the open-source world. ๐Ÿ’ก

Thank you for making an impact! ๐Ÿš€

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.

2 participants