-
Notifications
You must be signed in to change notification settings - Fork 3
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
Use trixi_include
from TrixiBase.jl
#88
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #88 +/- ##
==========================================
+ Coverage 96.12% 96.15% +0.02%
==========================================
Files 17 17
Lines 1111 1065 -46
==========================================
- Hits 1068 1024 -44
+ Misses 43 41 -2 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This requires your changes to TrixiBase.jl and we're fine, I guess?
Yes. So let's wait for trixi-framework/TrixiBase.jl#13. |
Co-authored-by: Hendrik Ranocha <[email protected]>
It looks like the availability of the new version has not reached all package servers yet. Maybe let's wait a bit and re-trigger CI after a few hours? |
I'm currently on my phone. So (I think) I cannot suggest any code change in the App, but the compat bound for TrixiBase.jl should be bumped in the docs directory, too. |
CI is passing now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Some utility functions moved to the new package TrixiBase.jl.