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

Do not @ giscus bot #238

Merged
merged 1 commit into from
Sep 16, 2024
Merged

Do not @ giscus bot #238

merged 1 commit into from
Sep 16, 2024

Conversation

glenn-jocher
Copy link
Member

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

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhance the response content by removing specific bot mentions. πŸ€–βœ¨

πŸ“Š Key Changes

  • Added a mechanism to filter out " @giscus[bot]" from API response content.

🎯 Purpose & Impact

  • Cleaner Outputs: Improves readability by removing unnecessary bot mentions from messages.
  • Better User Experience: Ensures users receive more relevant and concise information without distractions.

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

πŸ‘‹ Hello @glenn-jocher, thank you for submitting a fantastic PR to the ultralytics/actions repository! πŸš€ This is an automated response to help guide your submission, and an Ultralytics engineer will assist you shortly. Please review the checklist below to ensure everything is on track:

  • βœ… Define a Purpose: Clearly explain why you made these changes in the PR description. It's great to see you've already outlined the goal and impact. For reference, you can link to any relevant issues here.
  • βœ… Synchronize with Source: Ensure your branch is up-to-date with the main branch of the ultralytics/actions repository. You can do this by clicking 'Update branch' or running git pull and git merge main locally.
  • βœ… Ensure CI Checks Pass: Verify all Continuous Integration (CI) checks are passing. If any checks fail, make sure to resolve them.
  • βœ… Update Documentation: Check that relevant documentation is updated for the changes made.
  • βœ… Add Tests: If applicable, please include or update tests and ensure all tests are passing.
  • βœ… Sign the CLA: If this is your first PR with us, sign the Contributor License Agreement by stating "I have read the CLA Document and I sign the CLA" in a comment.
  • βœ… Minimize Changes: Keep changes minimal and focused on the intended fix. Remember Bruce Lee's wisdom: "It is not daily increase but daily decrease, hack away the unessential..."

If you need further direction, please check our Contributing Guide. Feel free to leave a comment if you have any questions. Thank you for your contribution! πŸ™Œ

@glenn-jocher glenn-jocher merged commit 385dc87 into main Sep 16, 2024
4 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch September 16, 2024 18:57
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