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 action.yml #273

Merged
merged 2 commits into from
Oct 31, 2024
Merged

Update action.yml #273

merged 2 commits into from
Oct 31, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Oct 31, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

This PR enhances the Ultralytics GitHub action by making the GitHub username and email configurable for commit operations.

πŸ“Š Key Changes

  • Introduced two new optional inputs: github_username and github_email.
  • Set default values for these inputs to "UltralyticsAssistant" and "web@ultralytics.com", respectively.
  • Updated the Git configuration step in the action to use these inputs instead of hardcoded values.

🎯 Purpose & Impact

  • Customization: Allows users to specify their preferred GitHub username and email for commit descriptions, rather than using the default values.
  • Flexibility: Enhances the action's adaptability for different users and workflows, leading to more personalized and descriptive commit histories.

@UltralyticsAssistant UltralyticsAssistant added devops GitHub Devops or MLops enhancement New feature or request labels Oct 31, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting an ultralytics/actions πŸš€ PR! This is an automated response to help guide you through the contribution process. An Ultralytics engineer will also assist you shortly. In the meantime, please review the checklist below to ensure everything is in order:

  • βœ… Purpose & Clarity: Please clearly explain the purpose of your changes in the PR description. Connect it to any relevant issues if applicable. Make sure your commit messages follow our project's conventions.
  • βœ… Branch Sync: Ensure your branch is up-to-date with the main branch of ultralytics/actions. If it's not current, you can update it by clicking the 'Update branch' button or using git pull and git merge main.
  • βœ… Pass CI Checks: Verify that all CI checks are green. Address any failing checks to make sure everything is working smoothly.
  • βœ… Documentation Updates: Update or add any necessary documentation in our documentation site for new or changed features.
  • βœ… Include Tests: If applicable, incorporate or modify tests for your changes and confirm they all pass successfully.
  • βœ… CLA Confirmation: Ensure you've signed our Contributor License Agreement if this is your first PR with us by stating "I have read the CLA Document and I sign the CLA" in a response comment.
  • βœ… Limit Scope: Keep changes limited to those necessary for the bug fix or feature enhancement. Remember Bruce Lee's wisdom: "It is not daily increase but daily decrease, hack away the unessential."

If you haven't already, please provide a minimum reproducible example (MRE) for any bug reports to help us understand the issue better.

Thank you for your contribution to Ultralytics! We're excited to see what you're working on! πŸ’ͺ✨

@glenn-jocher glenn-jocher merged commit e70f7f5 into main Oct 31, 2024
3 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch October 31, 2024 20:24
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