aleleba/effectcraft
1
0
Fork 0
mirror of https://github.com/storytold/effectcraft.git synced 2026-10-24 10:09:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2215 from dylanmtaylor/sonar/nosonar-placement

Browse Source 
fix: put the NOSONAR marker on the spawnSync line (SonarCloud follow-up)
This commit is contained in:
Dylan M. Taylor
2026-10-23 09:53:46 -04:00
committed by GitHub
parent 9066b64150 400fbb79e9
commit 16c30830ef
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
examples/check-ae-ease.mjs
+3 -3
View File
@@ -11,9 +11,9 @@ if (!process.argv[2] || !fs.statSync(cli, { throwIfNoEntry: false })?.isFile())
}
const tolerance = 1e-7;
function run(command, args) {
// NOSONAR: a developer-run comparison script; spawnSync never uses a shell and every command
// is either a fixed binary (`cargo`, `process.execPath`) or the CLI script path validated above.
const result = spawnSync(command, args, { cwd: root, windowsHide: true, encoding: 'utf8', maxBuffer: 8 * 1024 * 1024 });
// A developer-run comparison script; spawnSync never uses a shell and every command is either
// a fixed binary (`cargo`, `process.execPath`) or the CLI script path validated above.
const result = spawnSync(command, args, { cwd: root, windowsHide: true, encoding: 'utf8', maxBuffer: 8 * 1024 * 1024 }); // NOSONAR
if (result.status !== 0) throw new Error(result.stderr || result.stdout || 'Process failed');
return JSON.parse(result.stdout.replace(/^\uFEFF/, '').trim());
}