fix(node-18): hook tests

apache / cordova-lib

Apache Cordova Tooling Library

https://cordova.apache.org/

Apache License 2.0

221 stars 242 forks source link

fix(node-18): hook tests #916

Closed dpogue closed 1 year ago

dpogue commented 1 year ago

Please squash when merging! @erisu

Platforms affected

All

Motivation and Context

Tests were failing in node 18

Description

Add the hashbang #!/bin/sh line to shell hook scripts

Testing

All test passing in node 18 now

Checklist

[x] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[ ] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))

codecov-commenter commented 1 year ago

Codecov Report

Merging #916 (20e83de) into master (34a8a20) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #916   +/-   ##
=======================================
  Coverage   88.36%   88.36%           
=======================================
  Files          46       46           
  Lines        2123     2123           
=======================================
  Hits         1876     1876           
  Misses        247      247

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more