-
Notifications
You must be signed in to change notification settings - Fork 203
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
Sync nailgun on 2018-02-27 #293
Conversation
Is this the upgrade to the new nailgun version? |
Yes, this rebases our changes on the current master in facebook/nailgun. |
Have you double-checked this change locally and used it? I think we should dogfood it first before merging it, given how severe the upstream changes have been. |
I haven't tried it a lot, it seemed to work, and the nailgun tests seemed happy too. |
Thanks for taking care of this, I will battle test it locally to be sure our users are not affected. Will get back to this PR soon. |
2b06335
to
a3f5bfc
Compare
Co-authored-by: Duhemm <martin.duhem@gmail.com> Co-authored-by: Jorge Vicente Cantero <jorgevc@fastmail.es>
6e4fefa
to
47781c1
Compare
I have updated nailgun to a version that includes a fix for #209, then released the version and changed it in our build. Let's merge this when CI is green. |
LGTM 👍 |
No description provided.