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
CI Updates #1562
Comments
CI is currently broken for master, PR to fix: #1561 |
Added v11.0.0 to the matrix. Also added v6.0.0 because that's the oldest we support for v4, so should be tested. |
Had to change to v6.2.1 because NVS was having issues with 6.0.0 - probably because the zip/7z packages are not available. |
But besides that it's a great tool for this task, right? |
Haven't reviewed it carefully but it looks great 👍 |
@nodejs/node-gyp FYI, let me know if there's an issue with any of this.
In nodegyp-test-pull-request:
PR_ID
toREMOTE_REF
, this allows us to test anything invoking the multiple subjobs at once. The job name remains "test pull request", so the whole job tree could probably be reorganized to make more sense. For now, we can think of it as "not exclusively".In nodegyp-test-commit:
The text was updated successfully, but these errors were encountered: