fix: removes ?? for node compat (#1576)

43081j · web-flow · commit 2523cd72c942 · 2024-01-12T21:42:38.000+01:00
We shipped syntax which is beyond our `engine` constraint. `??` is
available in node 14 but 4.x is node 4 and above, while 5.x is node 12
and above.

This just dumbs it back down to `||` for now at least.
diff --git a/lib/chai/assertion.js b/lib/chai/assertion.js
@@ -54,7 +54,7 @@ export function Assertion (obj, msg, ssfi, lockSsfi) {
   util.flag(this, 'lockSsfi', lockSsfi);
   util.flag(this, 'object', obj);
   util.flag(this, 'message', msg);
-  util.flag(this, 'eql', config.deepEqual ?? util.eql);
+  util.flag(this, 'eql', config.deepEqual || util.eql);
 
   return util.proxify(this);
 }

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ export function Assertion (obj, msg, ssfi, lockSsfi) {`
`54`	`54`	`util.flag(this, 'lockSsfi', lockSsfi);`
`55`	`55`	`util.flag(this, 'object', obj);`
`56`	`56`	`util.flag(this, 'message', msg);`
`57`		`- util.flag(this, 'eql', config.deepEqual ?? util.eql);`
	`57`	`+ util.flag(this, 'eql', config.deepEqual \|\| util.eql);`
`58`	`58`
`59`	`59`	`return util.proxify(this);`
`60`	`60`	`}`