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 README.md #59

Merged
merged 3 commits into from
Jan 18, 2024
Merged

Update README.md #59

merged 3 commits into from
Jan 18, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 18, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Streamlined the PR Summary GitHub Action script fetching method.

πŸ“Š Key Changes

  • The original inline script execution from utils/run_pr_summary.py has been replaced.
  • Now the script is fetched directly from the GitHub repository using curl before execution.

🎯 Purpose & Impact

  • Enhanced Maintainability: Fetching the latest version of the script ensures that the action is always up-to-date with the repository's latest changes.
  • Robust Execution: Separating the download and execution steps may reduce potential errors within the action runtime.
  • User Transparency: Users can see a clear reference to the source of the script, increasing trust in the action's functionality.

@glenn-jocher glenn-jocher merged commit 507bca2 into main Jan 18, 2024
1 check passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch January 18, 2024 22:56
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

2 participants