-
Notifications
You must be signed in to change notification settings - Fork 31
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
Combined nightly fixes #127
Conversation
01cc0d1
to
ae1c5e7
Compare
Also disables some of the heavily nested AD tests for now, since they are causing the segfault.
ae1c5e7
to
68082ff
Compare
68082ff
to
496d032
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #127 +/- ##
==========================================
- Coverage 50.50% 48.06% -2.45%
==========================================
Files 27 27
Lines 2651 2657 +6
==========================================
- Hits 1339 1277 -62
- Misses 1312 1380 +68
... and 4 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ 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.
LGTM. Shall we merge this?
I think we should revisit the hack I added, but yeah, let's keep our CI green for now. |
This combines #125 and #126
(and is ontop of #123 that I merged earlier)