NC | should retry add parentheses

romayalon · romayalon · commit c905f5aae5b3 · 2024-09-24T17:49:45.000+03:00
Signed-off-by: Romy &lt;35330373+romayalon@users.noreply.github.com&gt;
diff --git a/src/util/native_fs_utils.js b/src/util/native_fs_utils.js
@@ -271,7 +271,7 @@ function should_retry_link_unlink(is_gpfs, err) {
     const should_retry_general = ['ENOENT', 'EEXIST'].includes(err.code);
     const should_retry_gpfs = [gpfs_link_unlink_retry_err, gpfs_unlink_retry_catch].includes(err.code);
     const should_retry_posix = [posix_link_retry_err, posix_unlink_retry_err].includes(err.message);
-    return should_retry_general || is_gpfs ? should_retry_gpfs : should_retry_posix;
+    return should_retry_general || (is_gpfs ? should_retry_gpfs : should_retry_posix);
 }
 
 ////////////////////////

Original file line number	Diff line number	Diff line change
`@@ -271,7 +271,7 @@ function should_retry_link_unlink(is_gpfs, err) {`
`271`	`271`	`const should_retry_general = ['ENOENT', 'EEXIST'].includes(err.code);`
`272`	`272`	`const should_retry_gpfs = [gpfs_link_unlink_retry_err, gpfs_unlink_retry_catch].includes(err.code);`
`273`	`273`	`const should_retry_posix = [posix_link_retry_err, posix_unlink_retry_err].includes(err.message);`
`274`		`- return should_retry_general \|\| is_gpfs ? should_retry_gpfs : should_retry_posix;`
	`274`	`+ return should_retry_general \|\| (is_gpfs ? should_retry_gpfs : should_retry_posix);`
`275`	`275`	`}`
`276`	`276`
`277`	`277`	`////////////////////////`