fix: remove comment lines

Jeff Gordon · Jeff Gordon · commit 6e1f7b9e55de · 2024-10-01T23:16:06.000-05:00
diff --git a/lib/uv.js b/lib/uv.js
@@ -89,16 +89,16 @@ function removeEditableFlagFromRequirementsString(requirementBuffer) {
   const flagStr = '-e ';
   const commentLine = '#';
   const lines = requirementBuffer.toString('utf8').split(EOL);
-  const newLines = [];
+  let updatedLines = [];
   for (let i = 0; i < lines.length; i++) {
     if (lines[i].startsWith(flagStr)) {
-      newLines.push(lines[i].substring(flagStr.length));
+      updatedLines.push(lines[i].substring(flagStr.length));
     }
-    if (lines[i].startsWith(commentLine)) {
-      continue;
+    if (!lines[i].startsWith(commentLine)) {
+      updatedLines.push(lines[i]);
     }
   }
-  return Buffer.from(newLines.join(EOL));
+  return Buffer.from(updatedLines.join(EOL));
 }
 
 module.exports = { uvToRequirements };

Original file line number	Diff line number	Diff line change
`@@ -89,16 +89,16 @@ function removeEditableFlagFromRequirementsString(requirementBuffer) {`
`89`	`89`	`const flagStr = '-e ';`
`90`	`90`	`const commentLine = '#';`
`91`	`91`	`const lines = requirementBuffer.toString('utf8').split(EOL);`
`92`		`- const newLines = [];`
	`92`	`+ let updatedLines = [];`
`93`	`93`	`for (let i = 0; i < lines.length; i++) {`
`94`	`94`	`if (lines[i].startsWith(flagStr)) {`
`95`		`- newLines.push(lines[i].substring(flagStr.length));`
	`95`	`+ updatedLines.push(lines[i].substring(flagStr.length));`
`96`	`96`	`}`
`97`		`- if (lines[i].startsWith(commentLine)) {`
`98`		`- continue;`
	`97`	`+ if (!lines[i].startsWith(commentLine)) {`
	`98`	`+ updatedLines.push(lines[i]);`
`99`	`99`	`}`
`100`	`100`	`}`
`101`		`- return Buffer.from(newLines.join(EOL));`
	`101`	`+ return Buffer.from(updatedLines.join(EOL));`
`102`	`102`	`}`
`103`	`103`
`104`	`104`	`module.exports = { uvToRequirements };`