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

Add Prettier for YAML formatting to Ultralytics Actions #2217

Merged
merged 2 commits into from
Jun 8, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 30, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved GitHub Actions configuration for better code formatting and validation.

πŸ“Š Key Changes

  • Changed the event trigger from pull_request_target to pull_request, enhancing security and workflow triggers.
  • Split YAML formatting into its own setting, now handled by prettier, emphasizing clearer structure and dedicated processing.

🎯 Purpose & Impact

  • Security & Reliability: Switching to pull_request from pull_request_target reduces security risks and aligns with best practices, ensuring that only trusted code triggers actions in a PR.
  • Code Quality: Separating YAML formatting with prettier ensures more specialized and appropriate formatting, improving readability and maintainability of configuration files.
  • User Experience: These changes make the development process smoother and safer, ultimately leading to a more robust and user-friendly application. Developers can expect more reliable automated formatting, reducing minor manual corrections and increasing focus on significant improvements.

Emoji guide 🧩:

  • πŸš€ Enhancing security and workflow efficiency.
  • ✨ Introducing specialized formatting for better code quality.
  • 😊 Improving developer and user experience with streamlined processes.

@glenn-jocher glenn-jocher merged commit 0d2eccf into master Jun 8, 2024
7 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch June 8, 2024 19:45
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.

None yet

1 participant