-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 #2308
Update format.yml #2308
Conversation
Signed-off-by: Glenn Jocher <[email protected]>
π Hello @glenn-jocher, thank you for submitting an
Thank you for the detailed PR description and summary! π The proposed changes to the GitHub Actions workflow sound great for improving security and maintaining compatibility with modern practices π. Please ensure all workflow updates have been validated to avoid any unintended issues upon merging. For more guidance, please refer to our Contributing Guide. This is an automated response π οΈ, but an Ultralytics engineer will review your PR shortly and provide additional feedback if needed. π |
π Hello @glenn-jocher, thank you for submitting an
For more guidance, please refer to our Contributing Guide. Donβt hesitate to leave a comment if you have any questions. Thank you for contributing to Ultralytics! π π A friendly note: this is an automated response to help you get started. An Ultralytics engineer will review your PR shortly and provide further feedback if needed. π |
π οΈ PR Summary
Made with β€οΈ by Ultralytics Actions
π Summary
Updated GitHub Actions to better handle pull request workflows for
yolov3
.π Key Changes
pull_request_target
withpull_request
in.github/workflows/format.yml
.opened
,closed
,synchronize
, andreview_requested
on themain
andmaster
branches.π― Purpose & Impact
pull_request
instead ofpull_request_target
helps limit potential security risks from untrusted code in forks.