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 publish.yml #109

Merged
merged 1 commit into from
Oct 29, 2024
Merged

Update publish.yml #109

merged 1 commit 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

Minor update to the Slack message format in the GitHub Actions workflow for successful builds.

πŸ“Š Key Changes

  • Adjusted the formatting of the Slack notification message sent after a successful GitHub Actions run by adding backticks around the repository name and package version.

🎯 Purpose & Impact

  • Enhanced Readability: The use of backticks improves the readability of the message by highlighting the repository name and package version. πŸ“
  • No Functional Changes: This update does not affect the functionality of the workflow or any user-facing features, making it a safe, cosmetic improvement. 🎨

@UltralyticsAssistant UltralyticsAssistant added the devops GitHub Devops or MLops label Oct 29, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting an update to the publish.yml file in the ultralytics/mkdocs repository! πŸš€ To help us make sure your contribution is integrated smoothly, please take a moment to review the checklist below:

  • βœ… Define a Purpose: Ensure the purpose of your update is clear in your PR description. Your formatting change in the Slack notification message is a great step towards clarity and enhanced communication within the team. Nice job! πŸ‘
  • βœ… Synchronize with Source: Please confirm your branch is up-to-date with the ultralytics/mkdocs main branch. You can easily update it by clicking the 'Update branch' button or pulling the latest changes locally and merging them into your branch.
  • βœ… Ensure CI Checks Pass: Make sure all Continuous Integration (CI) checks are passing. If you encounter any issues, please take the necessary steps to resolve them.
  • βœ… Update Documentation: Since your PR pertains to a script change, there's no need for additional documentation updates in this case. If future changes require documentation, you can find it here.
  • βœ… Add Tests: While this specific change might not require new tests, ensure any future functional modifications are accompanied by relevant tests.
  • βœ… Sign the CLA: If you haven't already, please sign the Contributor License Agreement by commenting "I have read the CLA Document and I sign the CLA" in this PR thread.
  • βœ… Minimize Changes: You've done well to keep the change minimal and focused, enhancing the message format without any unnecessary alterations. 🎯

For any questions, refer to our Contributing Guide or leave a comment. An Ultralytics engineer will also review and assist with your PR soon. πŸ› οΈ Thank you for helping make our repository better! 😊

@glenn-jocher glenn-jocher merged commit aa81f2a into main Oct 29, 2024
2 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch October 29, 2024 21:52
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