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 summarize_release.py #236

Merged
merged 3 commits into from
Sep 15, 2024
Merged

Update summarize_release.py #236

merged 3 commits into from
Sep 15, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Sep 15, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhancements to the release summary script for better efficiency and accuracy.

πŸ“Š Key Changes

  • Added a time.sleep(30) to ensure PR summaries are up-to-date after a merge.
  • Modified PR processing to sort pull requests from earliest to latest.

🎯 Purpose & Impact

  • πŸ•’ Ensures accurate data retrieval by allowing time for systems to update post-merge.
  • πŸ” Improves the order of PRs for clearer timeline presentation, enhancing readability.

@UltralyticsAssistant UltralyticsAssistant added the enhancement New feature or request label Sep 15, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting an ultralytics/actions πŸš€ PR! This is an automated response to help ensure a smooth integration of your changes. An Ultralytics engineer will also review your PR soon. Meanwhile, please go over the following checklist:

  • βœ… Define a Purpose: Ensure your PR description clearly explains the purpose of your updates and link to any relevant issues. Please verify that your commit messages are clear and follow the project's conventions.
  • βœ… Synchronize with Source: Make sure your branch is in sync with the ultralytics/actions main branch. If it's out of date, update it via the 'Update branch' button or by running git pull and git merge main locally.
  • βœ… Ensure CI Checks Pass: Confirm that all Ultralytics Continuous Integration (CI) checks have passed. Address any issues if the checks fail.
  • βœ… Update Documentation: Modify any relevant documentation if there are new features or changes.
  • βœ… Add Tests: Where applicable, include or update tests to cover your changes. Confirm all tests are passing.
  • βœ… Sign the CLA: If this is your first contribution to Ultralytics, please sign our Contributor License Agreement by commenting "I have read the CLA Document and I sign the CLA."
  • βœ… Minimize Changes: Keep your changes limited to what is necessary for your bug fix or feature addition. Remember, "It is not daily increase but daily decrease, hack away the unessential." β€” Bruce Lee

For further details, please check our Contributing Guide. Feel free to comment if you've got questions. Thank you for your contribution to Ultralytics! πŸš€πŸ˜Š

@glenn-jocher glenn-jocher merged commit 4cdfe88 into main Sep 15, 2024
3 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch September 15, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants