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

check_git_status() bug fix #1925

Merged
merged 1 commit into from
Jan 13, 2021
Merged

check_git_status() bug fix #1925

merged 1 commit into from
Jan 13, 2021

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 13, 2021

Fix for UnboundLocalError: local variable 's' referenced before assignment.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improved messaging for code updates in the YOLOv5 project.

📊 Key Changes

  • Updated the comment to suggest running 'git pull' in a more general context, not just for YOLOv5.
  • Simplified error handling by printing the exception directly instead of converting it to a string and then printing.

🎯 Purpose & Impact

  • The change to the recommendation language makes it applicable to more contexts, providing clearer guidance to users on how to keep their codebase up to date.
  • Streamlining the error output makes the code cleaner and potentially easier for users to understand the nature of any issues they encounter.

@glenn-jocher glenn-jocher merged commit 051e9e8 into master Jan 13, 2021
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1-1 branch January 13, 2021 18:25
KMint1819 pushed a commit to KMint1819/yolov5 that referenced this pull request May 12, 2021
taicaile pushed a commit to taicaile/yolov5 that referenced this pull request Oct 12, 2021
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
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.

1 participant