Commit 49be13c
tools: keep Emeriti lists case-insensitive alphabetic
The current code does not treat capital and lowercase letters as
equivalent.
PR-URL: #45159
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent ad9a5bb commit 49be13c
2 files changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -163,7 +163,7 @@ async function moveCollaboratorToEmeritus(peopleToMove) { | |||
| 163 | 163 | const currentLine = `${collaboratorFirstLine}\n${line}\n`; | |
| 164 | 164 | // If textToMove is empty, this still works because when undefined is | |
| 165 | 165 | // used in a comparison with <, the result is always false. | |
| 166 | - while (textToMove[0] < currentLine) { | ||
| 166 | + while (textToMove[0]?.toLowerCase() < currentLine.toLowerCase()) { | ||
| 167 | 167 | fileContents += textToMove.shift(); | |
| 168 | 168 | } | |
| 169 | 169 | fileContents += currentLine; | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -198,7 +198,7 @@ async function moveTscToEmeritus(peopleToMove) { | |||
| 198 | 198 | const currentLine = `${memberFirstLine}\n${line}\n`; | |
| 199 | 199 | // If textToMove is empty, this still works because when undefined is | |
| 200 | 200 | // used in a comparison with <, the result is always false. | |
| 201 | - while (textToMove[0] < currentLine) { | ||
| 201 | + while (textToMove[0]?.toLowerCase() < currentLine.toLowerCase()) { | ||
| 202 | 202 | fileContents += textToMove.shift(); | |
| 203 | 203 | } | |
| 204 | 204 | fileContents += currentLine; | |
0 commit comments