-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
Allow disabling of more features #3844
Conversation
Blah, silly me. It broke because it relies on something like #3846 so I'll get that fixed and landed first, then fixup this branch again and push a fix. |
15cdb98
to
bfb46bc
Compare
b54a97b
to
6618efb
Compare
I'm having a hard time figuring out why the windows CI builds fail like this, so I'm more or less bisecting in the separate branch bagder/ci. I might then land some of the commits piecemeal instead once proven to pass the CI. |
6618efb
to
07be034
Compare
Ah, here is the PR I was looking for. Did you forget a "Closes" line? These are in master now and broke the HTTP-only AppVeyor tests. |
Only some of them are in master, the ones I (wrongly apparently) thought didn't break master... |
07be034
to
1aa1e50
Compare
This is a patch set that