Skip to content

Commit 62c233c

Browse files
Trottdanielleadams
authored andcommitted
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 5aa4ea9 commit 62c233c

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

tools/find-inactive-collaborators.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ async function moveCollaboratorToEmeritus(peopleToMove) {
163163
const currentLine = `${collaboratorFirstLine}\n${line}\n`;
164164
// If textToMove is empty, this still works because when undefined is
165165
// used in a comparison with <, the result is always false.
166-
while (textToMove[0] < currentLine) {
166+
while (textToMove[0]?.toLowerCase() < currentLine.toLowerCase()) {
167167
fileContents += textToMove.shift();
168168
}
169169
fileContents += currentLine;

tools/find-inactive-tsc.mjs

+1-1
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ async function moveTscToEmeritus(peopleToMove) {
198198
const currentLine = `${memberFirstLine}\n${line}\n`;
199199
// If textToMove is empty, this still works because when undefined is
200200
// used in a comparison with <, the result is always false.
201-
while (textToMove[0] < currentLine) {
201+
while (textToMove[0]?.toLowerCase() < currentLine.toLowerCase()) {
202202
fileContents += textToMove.shift();
203203
}
204204
fileContents += currentLine;

0 commit comments

Comments
 (0)