Skip to content
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

investigate segfaults on CI #594

Closed
wants to merge 2 commits into from
Closed

investigate segfaults on CI #594

wants to merge 2 commits into from

Conversation

simeonschaub
Copy link
Collaborator

The segfaults on CI seem to point to these two tests in particular.
Let's see if disabling them fixes CI or whether that might have just
been a red herring

@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Attention: 71 lines in your changes are missing coverage. Please review.

Comparison is base (7beca92) 86.54% compared to head (6070f13) 82.26%.

Files Patch % Lines
src/builtins.jl 11.25% 71 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #594      +/-   ##
==========================================
- Coverage   86.54%   82.26%   -4.29%     
==========================================
  Files          12       12              
  Lines        2542     2458      -84     
==========================================
- Hits         2200     2022     -178     
- Misses        342      436      +94     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@simeonschaub
Copy link
Collaborator Author

Well, that certainly answers the question... 😅

@aviatesk
Copy link
Member

😱

@simeonschaub simeonschaub reopened this Nov 7, 2023
@simeonschaub
Copy link
Collaborator Author

Hmm, at least nightly doesn't segfault anymore, the tests just fail... 🤔

@aviatesk
Copy link
Member

How would you like to proceed with this issue? I'm of the view that for healthy development, it's acceptable to disable tests that are challenging to correct at the moment. My preference is to focus on restoring the CI as a first step.

@simeonschaub
Copy link
Collaborator Author

simeonschaub commented Nov 28, 2023

Yeah, unfortunately this PR showed that the issues ran deeper than just a single failing test. Wondering wether the segfaults might have to do with #591?

@simeonschaub simeonschaub deleted the sds/segfaults branch November 28, 2023 10:28
@aviatesk
Copy link
Member

Yeah, I will take a look on it later today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants