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

Integrate Continuous Testing Workflow #116

Merged
merged 7 commits into from Apr 11, 2024

Conversation

yogendrasinghx
Copy link
Member

@yogendrasinghx yogendrasinghx commented Apr 1, 2024

This pull request introduces a new GitHub Actions workflow for Continuous Testing (CT). The aim is to enhance code quality and reliability by automating tests on pull requests to the main branch.

Resolves: UHUB-71

I have read the CLA Document and I hereby sign the CLA

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Added Continuous Testing (CT) via GitHub Actions & Expanded Testing Framework for Hub SDK 🚀.

📊 Key Changes

  • Continuous Testing Workflow: Implemented a GitHub Actions workflow (ct.yml) for running tests on pull requests to the main branch, using Ubuntu latest and Python 3.x.
  • Testing Framework Enhancements: Added and modified test files to include fixtures for setup, test models, datasets, and projects handling, enriched with various utility functions.
  • Support Files: Included requirements and configuration (requirements.txt, pytest.ini) for the testing environment setup.

🎯 Purpose & Impact

  • Enhanced Quality Assurance: The introduction of Continuous Testing ensures every change is tested, increasing code reliability and stability.
  • Streamlined Testing Process: The expanded testing framework allows for more comprehensive and efficient testing strategies, covering a wider range of scenarios.
  • Developer Efficiency: These changes facilitate faster feedback loops for developers, improving overall productivity and code quality in the Hub SDK project.

hassaanfarooq01 and others added 6 commits March 23, 2024 20:24
Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
Co-authored-by: UltralyticsAssistant <web@ultralytics.com>
….3.2 in /.github/workflows (#112)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Resolves: UHUB-71
@glenn-jocher glenn-jocher self-requested a review April 11, 2024 17:05
@glenn-jocher
Copy link
Member

@yogendrasinghx @sergiuwaxmann this looks great! I see there is 1 failing test but it may be a transient failure.

In any case I think I'll merge this so as not to be a blocker, and we can raise a bug report for the failing test (re-running now to confirm status of the failing tes)

@glenn-jocher glenn-jocher merged commit 3fd4312 into main Apr 11, 2024
3 of 4 checks passed
@glenn-jocher glenn-jocher deleted the QA/UHUB-71/continuous-testing branch April 11, 2024 17:11
@glenn-jocher
Copy link
Member

@yogendrasinghx PR merged! Nice work here!!

This was referenced Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants