dpogue opened a new pull request, #281: URL: https://github.com/apache/cordova-js/pull/281
### Motivation and Context <!-- Why is this change required? What problem does it solve? --> <!-- If it fixes an open issue, please link to the issue here. --> Since we can't adopt the latest versions of execa and globby due to ESM, and we use them for very specific purposes here, replace them with built-ins where possible (and use fast-glob where not possible). ### Description <!-- Describe your changes in detail --> - Replace the single `execa` call with `child_process.spawnSync`, since we know that we're invoking `git` rather than a shell script - Replace the one spot that used `globby` with `fast-glob` - On versions of node with `fs.globSync`, use that instead of `fast-glob` - Fix a warning in a Jasmine test about `done` getting called with arguments - Update testing dependencies to latest versions ### Testing <!-- Please describe in detail how you tested your changes. --> - Confirmed that the contents of the cordova.js file are the same before and after these changes (some extra spaces removed, but no code changes) - Confirmed that all unit tests run and pass ### Checklist - [x] I've run the tests to see all new and existing tests pass -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org For additional commands, e-mail: issues-h...@cordova.apache.org
