fix: remove comment lines, again

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

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