-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
minor tweaks to CI test scripts #3467
Conversation
@alexlamsl Have you considered replacing travis-ci and appveyer with github workflows? Seems to be faster and more reliable. Supports windows and linux. https://help.github.com/en/articles/configuring-a-workflow Example: https://github.com/actions/setup-node/blob/master/.github/workflows/workflow.yml |
@kzc long time no see 😉 20 concurrent jobs definitely look promising compared with what we have at the moment - let me have a play with it next week. Will do it in a new PR as the feature is apparently in limited beta so just in case they pull the rug from underneath us. 😅 |
Glad to see you're still solving interesting puzzles.
With half the JS on the internet minified from this code base (or a fork of it), I think it wouldn't be an issue getting into the beta. |
Fair point, though I was concerned with potential discontinuation of Github Actions altogether. Just signed up - looks like it will only apply to projects that I directly own, so I'll get it working before bothering mishoo to sign up 👻 |
OT: Node.js 12 looks like it's gonna be a lot of fun...
Okay, let's run it again:
🤦 Edit: |
Where the heck did |
(voice of Paperclip) v8, do you mean The test case does trigger (& catch) a bunch of stack overflow errors, but I'm surprised it can corrupt memory like that. |
To be honest, with the sheer complexity of these multi-stage JITting tracing compilers with parallel generational garbage collection I'm surprised it doesn't happen more often. Got to hand it to fuzzing. If something can break - it'll find it. |
No description provided.