-
Notifications
You must be signed in to change notification settings - Fork 39
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
Node12+ref napi #327
base: master
Are you sure you want to change the base?
Node12+ref napi #327
Conversation
* upstream/master: (211 commits) NOJIRA: Fixed typo GPII-4231: Updating to version of universal with narrator autostart commented out GPII-3830: Mocking a wrapper fs.existsSync, rather than the real function. GPII-3830: Testing for unmounted usb drives GPII-4231: Adopted suggestion for mambo Revert "GPII-4428: Restoring icon spacing when restarting explorer." GPII-4428: Restoring icon spacing when restarting explorer. GPII-4231: Adding back one piece of mamba GPII-4231: Removing mamba GPII-4420: Handling the possibility of no installed language packs GPII-4420: Only listing languages installed by a morphic language installer GPII-4409: Improved test stability (increased timeout) GPII-4409: Refactored away null-array return of getProcessWindows GPII-4409: Launch buttons to activate existing application instance. GPII-4409: Getting the list of process windows, extracted from closeProcessByName for re-use. GPII-4382: Consistent with current universal master with fixed schemas GPII-4382: Added additional logging for SPI settings handler issues. GPII-4370: Rejecting settings handler if the settings helper fails. GPII-4382: Updated to use universal branch to test work in progress in CI. NOJIRA: Skipping service tests if running from electron. ...
* upstream/master: (110 commits) GPII-4428: Updating to current universal GPII-4470: Storing the currently active window in an environment variable. NOJIRA: Waiting for test window to show. GPII-3811: Made highcontrast theme test work GPII-3811: Fixed merge. GPII-4486 Updating universal dep GPII-4486 Forgot param portion of diff GPII-4486 Returning only the settings asked for in the payload GPII-4446: Updating to matching master Revert "GPII-3990: Using updated node-os-service." GPII-4428: Added API reference. Updating to current gpii-universal GPII-4397: Documented the userIdSource configuration. GPII-3853: Updated to matching universal release Revert "GPII-3853: Pointing back to the universal branch for metrics" GPII-3853: Pointing back to the universal branch for metrics GPII-4087: Removed windows.rm GPII-3853: Small tidy-up GPII-4446: Modifying the existing ribbon, rather than replacing it. GPII-4397: Fixed user login texts to recognised lower cased marker ...
CI job passed: https://ci.gpii.net/job/windows-tests/1461/ |
@@ -21,7 +21,7 @@ | |||
"json5": "2.1.0", | |||
"mkdirp": "0.5.1", | |||
"minimist": "1.2.0", | |||
"@gpii/os-service": "2.2.0", | |||
"@gpii/os-service": "javihernandez/node-os-service#GPII-4140", |
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.
What's in this one?
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.
It's an update with upstream/master, which adds support for node12, see: GPII/node-os-service#2
This is just a test to see if the replacement of ref with ref-napi produces the same problems that I'm getting locally on CI