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

OS and Python CI Matrix #1003

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

OS and Python CI Matrix #1003

wants to merge 1 commit into from

Conversation

glenn-jocher
Copy link
Member

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

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Expanded CI testing matrix to improve compatibility across more operating systems and Python versions. πŸ› οΈπŸ”

πŸ“Š Key Changes

  • Added additional OS support: macOS-15, Windows-latest, and Ubuntu-24.04-arm.
  • Extended Python version support to include 3.8 through 3.13.
  • Increased CI job timeout to 180 minutes for comprehensive testing.
  • Limited parallel jobs with max-parallel: 1 to ensure stable test execution.

🎯 Purpose & Impact

  • Broader OS Coverage: Ensures the project works seamlessly across popular OS platforms, including macOS and new Ubuntu versions. πŸ–₯️
  • Enhanced Python Compatibility: Supports a wider range of Python environments for greater user flexibility. 🐍
  • Reliable Testing: Longer testing periods and serialized test execution improve stability and accuracy in CI checks. βœ…
  • Developer Confidence: Bolsters trust in the software's functionality across diverse environments. πŸš€

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@UltralyticsAssistant UltralyticsAssistant added the devops GitHub Devops or MLops label Jan 30, 2025
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting your ultralytics/hub πŸš€ PR! To ensure a smooth and efficient review process, please go through the following checklist:

  • βœ… Define a Purpose: Provide a clear explanation of your CI matrix update in the PR description. Link any related issues if applicable, and ensure your commit messages adhere to the repository's guidelines.
  • βœ… Synchronize with Source: Confirm the branch for your PR is up-to-date with the latest ultralytics/hub main branch. Use the 'Update branch' button or synchronize manually with git pull and git merge main.
  • βœ… Verify CI Checks: Ensure all Ultralytics Continuous Integration (CI) checks pass successfully. If you encounter any failures, please address them before requesting additional reviews.
  • βœ… Focus Documentation: Update or verify any relevant documentation to support the changes made in the expanded CI matrix.
  • βœ… Test Changes: Add or update any necessary tests to validate CI configurations, ensuring all tests execute successfully.
  • βœ… Sign the CLA: If this is your first contribution, make sure you've signed the Contributor License Agreement by commenting with "I have read the CLA Document and I sign the CLA."
  • βœ… Minimize Changes: Limit the scope of your changes to focus directly on your intent: enhancing the CI matrix. As noted, "It is not daily increase but daily decrease, hack away the unessential." β€” Bruce Lee πŸ₯‹

Your PR aims for broader test coverage, which is fantastic πŸŽ‰! We'd like you to ensure that all changes to the .github/workflows/ci.yml configuration abide by best practices to maintain reliability across all environments.

For additional guidance, our Contributing Guide is a great resource. Should you have any questions or need assistance, feel free to comment here. An Ultralytics engineer will review your submission and assist as soon as possible. Thank you for your excellent contribution to 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants