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

Update pyproject.toml #36

Merged
merged 12 commits into from
Oct 29, 2024
Merged

Update pyproject.toml #36

merged 12 commits into from
Oct 29, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Oct 29, 2024

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

This PR introduces continuous integration (CI) to ensure code quality and updates various project configurations and documentation.

📊 Key Changes

  • CI Workflow Added: A GitHub Actions workflow for CI with tests on multiple OS (Ubuntu, macOS, Windows) and Python versions (3.12, 3.13).
  • Dependency Update: Increased setuptools version requirement from >=43.0.0 to >=70.0.0.
  • Modular Enhancements: Modified add_numbers to support tensor addition and added a main function for execution.
  • Testing Improvements: Updated test coverage for add_numbers to include both positive and negative cases.
  • CLI Command: Added a command line entry point example-cli-command.

🎯 Purpose & Impact

  • Automated Testing: Ensures that code changes do not break existing functionality across different environments, enhancing reliability and robustness. 🛡️
  • Documentation and Structure: Improved usability with clearer function documentation and example code execution paths. 📝
  • Developer Experience: Simplifies contribution and testing processes through the CI workflow and updated dependencies, encouraging more consistent development practices. 🚀
  • Broader Functionality: Extends mathematical operations with support for tensors, catering to more advanced data processing needs. 📈

@UltralyticsAssistant UltralyticsAssistant added dependencies Dependencies and packages enhancement New feature or request labels Oct 29, 2024
@UltralyticsAssistant
Copy link
Member

👋 Hello @glenn-jocher, thank you for submitting a PR to the ultralytics/template repository! 🚀 This is an automated response to assist you in ensuring a smooth integration for your updates. An Ultralytics engineer will also review your submission soon. Please review the following checklist:

  • Define a Purpose: Clearly explain the purpose of your fix or feature in your PR description, and link to any relevant issues if applicable. Make sure your commit messages are clear and follow the project's conventions.
  • Synchronize with Source: Make sure your PR is up-to-date with the ultralytics/template main branch. If it's not, you can update your branch by clicking the 'Update branch' button on GitHub or running git pull and git merge main locally.
  • Ensure CI Checks Pass: Check that all Ultralytics Continuous Integration (CI) checks are passing. If any checks fail, please address the issues promptly.
  • Update Documentation: Update the relevant documentation for any new or altered features, to ensure users benefit from your improvements.
  • Add Tests: If applicable, add or update tests to cover your changes and verify that all tests are passing.
  • Sign the CLA: Make sure you have signed our Contributor License Agreement. If it's your first PR, include "I have read the CLA Document and I sign the CLA" in a new comment.
  • Minimize Changes: Keep your changes to the essential minimum required for your update. "It is not daily increase but daily decrease, hack away the unessential." — Bruce Lee

For additional guidance, please see our Contributing Guide. Feel free to comment here if you have any questions or need further assistance. Thanks for your contribution and welcome aboard the Ultralytics team! 🚀💪

@glenn-jocher glenn-jocher merged commit 3c20674 into main Oct 29, 2024
9 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch October 29, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependencies and packages enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants