fix: remove logical assignment operator

[n1ru4l]

Hey, a user of one of our libraries reported an unexpected syntax error on Node.js 14, which I tracked down to this line.

The package.json of this project lists "node": ">=12.18" in the engines.

Logical Assignment is only supported since Node.js 15.14.

To me, it seems like this usage here got added unexpectedly.

---

We would love to prevent such things in the future by setting up a GitHub action that runs eslint with the following rule: https://github.com/weiran-zsd/eslint-plugin-node/blob/HEAD/docs/rules/no-unsupported-features/es-syntax.md

Would that be something you are interested in?

[codecov-commenter]

Codecov Report

Merging #1318 (dab5f36) into main (8b1dcd4) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1318   +/-   ##
=======================================
  Coverage   94.18%   94.18%           
=======================================
  Files          45       45           
  Lines        4125     4125           
=======================================
  Hits         3885     3885           
  Misses        240      240           

Impacted Files	Coverage Δ
lib/fetch/index.js	80.29% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b1dcd4...dab5f36. Read the comment docs.

[ronag]

Thanks!

[dotansimha]

@ronag any planned release soon with this fix?

[matthewlilley]

088518a

[dominykas]

Are you sure this is the only issue? Right now, the fetch tests are explicitly disabled - and just enabling them does not make them run in Node.js 14 - I'm getting a bunch of errors, starting with require('util/types') (only require('util').types is available in node 14), then cimplaints about AbortController (experimental in 14), etc

[ronag]

Fetch does not work on node 14. This is expected.