-
Notifications
You must be signed in to change notification settings - Fork 726
managed to drop a space in cabal-install version #11372
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
f887be4 to
c9532b0
Compare
07e0383 to
ad01460
Compare
|
Next push I'll remove the debugging stuff and put it back in the debugging PR, since I've proved I didn't make things (half-)work with all the hackery in the debug PR's |
ad01460 to
c9532b0
Compare
|
I'll finish debugging on the debug PR. I do note that the last build seems to have suggested that somehow |
|
Also, that's two network-related failures so far on the latest build. GitHub's evidently having a good day…. |
|
Got it, testing on test PR, will add that change here and re-label afterward. That still leaves #11340: if our CI actually is including it, what if anything is being done differently there? /cc: @AndreasPK |
|
Someone might want to re-review given the flags fix. |
Merge Queue Status✅ The pull request has been merged at ddf7a65 This pull request spent 10 minutes 48 seconds in the queue, with no time running CI. Required conditions to merge
|
I swear I tested the previous fix before pushing…
Space re-added. (Still need to figure out why the
Cabalgit information is missing.)Template B: This PR does not modify behaviour or interface
E.g. the PR only touches documentation or tests, does refactorings, etc.
Include the following checklist in your PR: