8000 Robust summarize_release.py previous tag by glenn-jocher · Pull Request #475 · ultralytics/actions · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Robust summarize_release.py previous tag #475

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Jun 4, 2025
Merged

Robust summarize_release.py previous tag #475

merged 7 commits into from
Jun 4, 2025

Conversation

glenn-jocher
Copy link
Member
@glenn-jocher glenn-jocher commented Jun 4, 2025 •
edited by UltralyticsAssistant
Loading

@asabri97 may fix publish issues with YOLO Flutter App repo

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improves the release summary workflow by making it more robust against API errors and edge cases, with enhanced error handling and expanded tests. 🚀

📊 Key Changes

  • Adds checks for failed API responses and missing commit data when comparing tags.
  • Refines logic for determining the previous Git tag, with better fallback handling.
  • Updates and expands tests to cover API failures, missing data, and empty commit lists.
  • Bumps the package version from 0.0.85 to 0.0.86.

🎯 Purpose & Impact

  • Ensures the release summary process is more reliable, even if the GitHub API returns errors or incomplete data.
  • Reduces the risk of crashes or incorrect summaries during release automation.
  • Provides better test coverage, making future changes safer and easier to maintain.
  • Users and developers can expect smoother, more dependable release notes generation. 📝✨

8000
@glenn-jocher
Robust summarize_release.py previous tag
919b5b1 
@asabri97 may fix publish issues with YOLO Flutter App repo

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher
Update __init__.py
a1664c9 
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops enhancement New feature or request labels Jun 4, 2025
@UltralyticsAssistant
Copy link
Member

👋 Hello @glenn-jocher, thank you for submitting an ultralytics/actions 🚀 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 link to any relevant issues. Ensure your commit messages are clear, concise, and follow the project's conventions.
  • ✅ Synchronize with Source: Confirm your PR is synchronized with the ultralytics/actions main branch. If it's behind, update it by clicking the 'Update branch' button or by running git pull and git merge main locally.
  • ✅ Ensure CI Checks Pass: Verify all Ultralytics Continuous Integration (CI) checks are passing. If any checks fail, please address the issues.
  • ✅ Update Documentation: Update the relevant documentation for any new or modified features.
  • ✅ Add Tests: If applicable, include or update tests to cover your changes, and confirm that all tests are passing.
  • ✅ Sign the CLA: Please ensure you have signed our Contributor License Agreement if this is your first Ultralytics PR by writing "I have read the CLA Document and I sign the CLA" in a new message.
  • ✅ Minimize Changes: Limit your changes to the minimum necessary for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." — Bruce Lee

For more guidance, please refer to our Contributing Guide. Don’t hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! 🚀

This is an automated response — an Ultralytics engineer will assist you soon.

@codecov Codecov
Copy link
codecov bot commented Jun 4, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 79.31034% with 6 lines in your changes missing coverage. Please review.

Project coverage is 75.03%. Comparing base (cde8423) to head (a729404).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
actions/summarize_release.py 53.84% 6 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #475      +/-   ##
==========================================
- Coverage   75.07%   75.03%   -0.04%     
==========================================
  Files          24       24              
  Lines        1404     1418      +14     
==========================================
+ Hits         1054     1064      +10     
- Misses        350      354       +4

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@glenn-jocher glenn-jocher merged commit f57d151 into main Jun 4, 2025
11 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch June 4, 2025 00:49
@UltralyticsAssistant
Copy link
Member

Fantastic work, @glenn-jocher! 🚀 "Quality means doing it right when no one is looking." – Henry Ford. Your thoughtful improvements to the release summary workflow embody this spirit, making Ultralytics Actions more resilient and reliable for everyone. Your attention to detail and commitment to robust automation are truly appreciated—thank you for making a real difference!

@glenn-jocher glenn-jocher mentioned this pull request Jun 4, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
devops GitHub Devops or MLops enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@glenn-jocher @UltralyticsAssistant
0