-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Run setup.py develop inside of PEP 518 build environment #4999
Conversation
I'm closing this because it seems to have caused unnecessary confusion about gh-4799. |
Hello! I am an automated bot and I have noticed that this pull request is not currently able to be merged. If you are able to either merge the |
cc @pradyunsg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks OK to me. Can you fix the lint issue (unused result var in the test) and add a news file? Hopefully PyPy3 will complete when the fixes are submitted.
@pfmoore pypy has timed out (again). |
Restarted.
…On Fri, 2 Mar 2018, 01:18 xoviat, ***@***.***> wrote:
@pfmoore <https://github.com/pfmoore> pypy has timed out (again).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4999 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ADH7SZdsAmR0TnrHmSthwSjkk6VuBiyxks5taFCUgaJpZM4Rv5UP>
.
|
I'm getting very sick of the PyPy3 checks :-( @pradyunsg @dstufft what do you think about making that check optional (like Python nightly)? Is PyPy 3 a critical platform for us? |
@pradyunsg Could you give this a quick check before I merge? I'd appreciate your confirmation as it's PEP 518 related. |
Wow, this is lightning speed! |
Small changes, easy to review 😉 |
I'm happy to make them optional for a short term fix. I want them to be run regularly and that we don't ignore their failures though.
+1 |
Ideally, we should fix them but I'm pretty sure no one has the bandwidth right now, so, ignoring failures is fine. I'll make a PR for this. |
Don't look at this until gh-4799 is merged.