-
Notifications
You must be signed in to change notification settings - Fork 88
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
Checker Error Message #792
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #792 +/- ##
==========================================
- Coverage 43.73% 43.72% -0.01%
==========================================
Files 65 65
Lines 19042 19035 -7
Branches 2324 2331 +7
==========================================
- Hits 8328 8324 -4
+ Misses 9307 9304 -3
Partials 1407 1407 ☔ View full report in Codecov by Sentry. |
This was intentional. I don't like the use of long strings of characters because how they appears will depend on the width of your terminal and its font. It seems very old school and somehow bothers me. It also makes it harder to check for such patterns in the future, since I do it via |
Thanks for teaching me good coding practices! I've moved the |
Description
Makes case file error messages clearer and more obvious
Type of change