-
Notifications
You must be signed in to change notification settings - Fork 1
Test: added allocations tests for types that are known to be allocation-free (SKIP THEM FOR NOW) #187
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #187 +/- ##
=======================================
Coverage 98.88% 98.88%
=======================================
Files 25 25
Lines 4205 4205
=======================================
Hits 4158 4158
Misses 47 47 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Also skip continue-on-error: ${{ matrix.version == 'nightly' || matrix.version == 'pre' }} For now the tests on the |
The tests pass if Will skip the allocations test for now with |
I choose to use the builtin
@allocations
macro for two reasons:@ballocations
in BechmarkToolsAs a drawback there is known limitations (or feature? 😄) for exemple JuliaLang/julia#51112. A simple workaround for this specific issue is to create another function that calls the tested in-place function and returns
nothing
:It seems to work well on Julia 1.11.