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 general.py #9252

Merged
merged 1 commit into from
Sep 1, 2022
Merged

Update general.py #9252

merged 1 commit into from
Sep 1, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Sep 1, 2022

Signed-off-by: Glenn Jocher glenn.jocher@ultralytics.com

๐Ÿ› ๏ธ PR Summary

Made with โค๏ธ by Ultralytics Actions

๐ŸŒŸ Summary

Improvement of error messaging with emojis for better user experience in version checks.

๐Ÿ“Š Key Changes

  • Emojis are now added to error messages during version checks when requirements are not met.

๐ŸŽฏ Purpose & Impact

  • Purpose: To make error messages more user-friendly and noticeable, especially when there's a version mismatch or minimum requirements are not met.
  • Impact: Users will experience more engaging and clear communication, which may improve debugging and upgrading experiences. This change is minor but enhances user interaction with the software. ๐Ÿ› ๏ธ๐Ÿ’ฌ

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher merged commit ea98199 into master Sep 1, 2022
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-2 branch September 1, 2022 17:36
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
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