-
Notifications
You must be signed in to change notification settings - Fork 352
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
Remove --allow-external #1198
Remove --allow-external #1198
Conversation
ad0962e
to
e0d385d
Compare
CI fail. |
It could need the same fix as I did in Fuel (define an env variable MKL_THREADING_LAYER=GNU) |
Fixed |
2 CI still fail. I think a new version of pep8 is used. That caused a failure in Fuel |
Ok, I figured out the problem. Virtualenv puts a bunch of its things into |
@nouiz I give up. I was tinkering for quite a while. I fixed Travis and Scrutinizer tests. But I have no idea what happens to Scrutinizer analysis. |
So, what do we do? As the tests passed, we could merge. This would fix possible security issue, so I would go in that direction. |
OK, let me cleanup history then
…On Thu, Sep 6, 2018, 02:10 Frédéric Bastien ***@***.***> wrote:
So, what do we do? As the tests passed, we could merge. This would fix
possible security issue, so I would go in that direction.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1198 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACvVJ4aSpIr3aPKrDoVCYyhSFRVs042qks5uYDaxgaJpZM4WSLx3>
.
|
d6e45d0
to
05bd45f
Compare
05bd45f
to
c79f01f
Compare
For mila-iqia/fuel#405
@nouiz