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

Delete .github/workflows/greetings.yml #2286

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

UltralyticsAssistant
Copy link
Member

@UltralyticsAssistant UltralyticsAssistant commented Sep 12, 2024

This PR deletes the .github/workflows/greetings.yml file.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

The "Greetings" GitHub Actions workflow was removed from the YOLOv3 repository.

πŸ“Š Key Changes

  • Deleted the .github/workflows/greetings.yml file responsible for welcoming new contributors and providing initial guidance.

🎯 Purpose & Impact

  • Simplification: Reduces workflow complexity by removing automated greeting messages.
  • Reduced Overhead: Less administrative burden maintaining this particular workflow, allowing focus on core developments.

@UltralyticsAssistant
Copy link
Member Author

πŸš€ PR Review Request

Thank you, @UltralyticsAssistant, for your pull request to the YOLOv3 repository! πŸŽ‰

πŸ“„ PR Summary

You've proposed the removal of the .github/workflows/greetings.yml file. This aims to streamline communications and reduce notifications by removing automated greetings for new PRs and issues.

πŸ” Review Process

A human reviewer will look over your changes soon and provide additional feedback or approval. In the meantime, here are a few guidelines that can help improve future contributions:

πŸ“‘ Best Practices for PR Submission:

  1. Keep It Focused: Ensure that your pull request is limited to a single purpose or feature. It should not combine multiple unrelated changes.

  2. Stay Updated: Frequently update your branch with the base repository to resolve any conflicts and ensure seamless integration. You can easily do this by running:

    git pull upstream master
  3. Test Thoroughly: Make sure all tests pass locally and validate that your changes work as expected. Add new tests if your changes need them!

  4. Detailed Descriptions: Provide clear summaries of the changes, their impact, and any relevant motivations or context. Screenshots or GIFs help if UI changes are involved.

  5. Follow the Template: Use the provided pull request template for structure and completeness.

πŸ—‚οΈ Resources

  • YOLOv3 CI Status: Keep an eye on the continuous integration checks. It helps ensure all submissions meet the quality standards.

If you have any questions or need assistance, please feel free to ask. Thank you for contributing to the YOLOv3 project! πŸ˜ƒ


This process helps ensure smooth collaboration and maintains the quality of the codebase. Happy coding! πŸ–₯οΈπŸ”§

@glenn-jocher glenn-jocher merged commit 854394c into master Sep 12, 2024
9 checks passed
@glenn-jocher glenn-jocher deleted the delete-greetings-1726161016 branch September 12, 2024 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants