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 format.yml #234

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Update format.yml #234

merged 1 commit into from
Dec 23, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Dec 23, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhanced GitHub Actions token handling for workflow automation. πŸš€

πŸ“Š Key Changes

  • Updated the token configuration in .github/workflows/format.yml to use a fallback token (GITHUB_TOKEN) if _GITHUB_TOKEN is unavailable.

🎯 Purpose & Impact

  • βœ… Improved reliability: Ensures the workflow runs smoothly even if the custom _GITHUB_TOKEN is not set.
  • πŸ”§ Streamlined maintenance: Reduces the chance of workflow failures due to missing token secrets.
  • πŸ› οΈ User-friendly automation: Provides seamless automation for developers contributing to the project.

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops enhancement New feature or request labels Dec 23, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting an ultralytics/hub-sdk πŸš€ 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 if applicable. Ensure your commit messages are clear, concise, and adhere to the project's conventions.
  • βœ… Synchronize with Source: Confirm your PR is synchronized with the ultralytics/hub-sdk main branch. If it's behind, merge the latest changes from main to ensure compatibility.
  • βœ… Ensure CI Checks Pass: Verify all Continuous Integration (CI) checks are passing. If any checks fail, please address the issues promptly.
  • βœ… Update Documentation: Update any relevant documentation for new or modified features.
  • βœ… Add Tests: Include or update tests to cover your changes, and confirm that all tests are passing.
  • βœ… Sign the CLA: If this is your first Ultralytics PR, sign the Contributor License Agreement (CLA) by confirming in the PR that you agree to the terms.
  • βœ… Minimize Changes: Limit your modifications to the minimum necessary for achieving the purpose of the PR. "It is not daily increase but daily decrease, hack away the unessential." β€” Bruce Lee

Please note this is an automated response to guide you through the contribution process. An Ultralytics engineer will review and provide personalized feedback soon. Thank you for contributing to Ultralytics! πŸš€

@glenn-jocher glenn-jocher merged commit 7623cdc into main Dec 23, 2024
7 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch December 23, 2024 20:44
@UltralyticsAssistant
Copy link
Member

πŸŽ‰ PR Merged! πŸŽ‰ Huge thanks to @glenn-jocher for the thoughtful enhancement to our GitHub Actions workflow! πŸš€

Your update ensures smoother automation for everyone and showcases the power of reliable, user-friendly solutions. As Albert Einstein once said, "Strive not to be a success, but rather to be of value." Your contribution embodies that vision by making a real impact on developer experience here.

We deeply appreciate your effort and dedicationβ€”keep being awesome! 🌟

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops GitHub Devops or MLops enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants