-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
DOC: Small modification of error message #2956
Conversation
@pubpub-zz what is |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2956 +/- ##
=======================================
Coverage 96.36% 96.36%
=======================================
Files 52 52
Lines 8744 8744
Branches 1589 1589
=======================================
Hits 8426 8426
Misses 190 190
Partials 128 128 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
The name is |
It is obvious! I was reading it like a word, and not making the connection. Using "default" instead, would be marginally better? Unsure if good practice to do so. |
The official name inside the stdlib is |
Right. So would be okay to change? Or are you saying because of this it would not be okay to change? |
It is internal and improves readability, thus changing it is possible and makes sense to avoid doubts. |
Also slight improvement of two error messages.
To increase readability. Also slight improvement of two error messages.
I prefered not to use |
No description provided.