Fix command substitution issue for Windows OS. (#677)

* fix windows script build

* update node version

* update README to reference CHANGELOG

.github/workflows/pr_test.yml

@@ -61,7 +61,7 @@ jobs:
       matrix:
         tinygo-version: [0.24.0]
         go-version: [1.18.x]
-        node-version: [12]
+        node-version: [18]
         rust-toolchain: [stable]
         platform: [ubuntu-latest, macos-latest, windows-latest]
     runs-on: ${{ matrix.platform }}

README.md

@@ -29,6 +29,8 @@ using the bug report template.
 
 [bug]: https://github.com/fastly/cli/issues/new?labels=bug&template=bug_report.md
 
+Please also check the [CHANGELOG](./CHANGELOG.md) for any breaking-changes or migration guidance.
+
 ### Security issues
 
 Please see our [SECURITY.md](SECURITY.md) for guidance on reporting security-related issues.

pkg/commands/compute/language_javascript.go

@@ -27,7 +27,7 @@ const JsCompilationURL = "https://www.npmjs.com/package/@fastly/js-compute"
 // are simply upgrading their CLI version and might not be familiar with the
 // changes in the 4.0.0 release with regards to how build logic has moved to the
 // fastly.toml manifest.
-const JsDefaultBuildCommand = "$(npm bin)/webpack && $(npm bin)/js-compute-runtime ./bin/index.js ./bin/main.wasm"
+const JsDefaultBuildCommand = "npm exec webpack && npm exec js-compute-runtime ./bin/index.js ./bin/main.wasm"
 
 // JsInstaller is the command used to install the dependencies defined within
 // the Js language manifest.