-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Disable travis [take 526] #23887
Comments
I fully agree. |
It happens on stdenv rebuild. Our documentation depends on pandoc, which takes ages to build and travis-ci can't handle it. |
I think this time it's that nixpkgs tarball job (or specifically eval-release.nix) takes more memory than is available in Travis. |
The specifics seem less important than the gist of what I'm saying. I largely just ignore the emails because I don't trust the system to do anything useful for me. Does anyone get any value out of it, and enough value to justify the amount of noise it causes for everyone else? |
+1million. |
Shockingly I'm still getting bullshit emails. Now that hydra can build PRs I think we really need to just kill this. |
Let's just reduce it to checking evaluation only. |
Even that breaks spuriously at times. Can you honestly say you've ever gotten value out of these travis checks? |
Fixed by d0b4c5f |
I cna say that I have got some value out of Travis build logs when reviewing some PRs, but still the situation is ridiculous. |
I know we've raised this before, but it's getting ridiculous. Every email I get from it is "still broken", and everything in the commit log for the past few days has been a red X. It's providing negative value right now, and I think we should just turn it off until someone has time to put against fixing it or replacing it with a more tailored solution.
@edolstra @shlevy @domenkozar @grahamc
The text was updated successfully, but these errors were encountered: