-
Notifications
You must be signed in to change notification settings - Fork 73
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
fix: execvp: printf: Argument list too long #147
Conversation
Please run psf/black on the code so that the Python tests pass. |
This comment was marked as outdated.
This comment was marked as outdated.
Could we split the changes into one commit for each file and more meaningful messages? For
With ideally some text that explains the purpose of the new variable. For
For |
I have created #152 and #151, I will leave this for make when the other two merged. |
@danbev Not sure the commit msg is accure, PTAL. |
That looks good to me 👍 |
fix #144
fix nodejs/node#40735
cc @targos @danbev