-
Notifications
You must be signed in to change notification settings - Fork 58
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
import Base.trunc
#1536
import Base.trunc
#1536
Conversation
Hmm, tests fail in an (to me) unexpected way:
|
Locally, I get the error also in Nemo's current master branch:
|
See #1537 |
b502a2c
to
c0886ac
Compare
Codecov Report
@@ Coverage Diff @@
## master #1536 +/- ##
==========================================
- Coverage 84.03% 84.02% -0.01%
==========================================
Files 94 94
Lines 36613 36613
==========================================
- Hits 30767 30765 -2
- Misses 5846 5848 +2
... and 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
The required test
|
Without this change,
Base.trunc
is different fromNemo.trunc
, and the following happens.(I noticed this problem when a function for tab completion ran over the variables in
Main
, which triggered the warning.)