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 #272

Merged
merged 1 commit into from
Oct 29, 2024
Merged

Update publish.yml #272

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 tweak to the Slack notification format in the GitHub Actions workflow.

πŸ“Š Key Changes

  • Adjusted the Slack message format slightly by changing quotation marks around a repository variable to backticks.

🎯 Purpose & Impact

  • Improved Readability: The change enhances the visual clarity of messages sent to Slack, making it easier for users to identify the repository information.
  • Consistency: Maintains a uniform format for inline code elements across messages, helping teams quickly parse details from automated notifications.

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

πŸ‘‹ Hello @glenn-jocher, thank you for submitting a PR to ultralytics/actions! πŸš€ This is an automated response, but rest assured an Ultralytics engineer will assist you soon. In the meantime, to help us integrate your work efficiently, please review the checklist below:

  • βœ… Clearly Define Your Purpose: Please ensure your PR description clearly conveys the purpose of your update, and link to any relevant issues. This helps in understanding the context and impact of your changes.
  • βœ… Stay in Sync: Make certain your branch is up-to-date with the main branch of ultralytics/actions. If it’s behind, synchronizing is crucial. You can update by clicking the 'Update branch' button or running git pull and git merge main locally.
  • βœ… Pass All CI Checks: Verify that all Ultralytics Continuous Integration (CI) checks are passing. If any checks are unsuccessful, addressing them promptly is essential.
  • βœ… Document Updates: If your change affects functionality, ensure relevant documentation is updated to reflect these changes.
  • βœ… Implement Tests: If applicable, include or update the necessary tests to cover your changes, and make sure all tests are passing.
  • βœ… Sign the CLA: If you haven’t already, please sign our Contributor License Agreement. You can do so by writing "I have read the CLA Document and I sign the CLA" in a new message within this PR.
  • βœ… Keep It Minimal: Strive to limit your changes to the minimum required for your purpose to make the review process more manageable.

For further instructions or to familiarize yourself with our processes, please check out our Contributing Guide. Feel free to leave questions or comments in this PR thread. Thank you for your contribution to Ultralytics! πŸŽ‰

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