Skip to content

Commit 212111c

Browse files
committed
🚨 object-curly-spacing fix
1 parent 2417b39 commit 212111c

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

‎.eslintrc.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,6 @@ module.exports = {
1111
},
1212
rules: {
1313
"arrow-spacing": "error",
14+
"object-curly-spacing": ["error", "always"]
1415
},
1516
};

‎scripts/check-delegates-test.mjs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@ const threeLetter = `Chris de Almeida (CDA)\nRob Palmer (ROBPALMER)\nUjjwal Shar
99
const duplicate = `Chris de Almeida (CDA)\nRob Palmer (RPR)\nUjjwal Sharma (USA)\nUjjwal Sharma (USA)`;
1010
const valid = `Chris de Almeida (CDA)\nMichael Ficarra (MF)\nRob Palmer (RPR)\nUjjwal Sharma (USA)`;
1111

12-
assert.throws(() => checkDelegates(lex), { message: 'Line 3: Not in lexicographic order.'}); // also validates expected line number
13-
assert.throws(() => checkDelegates(missing), { message: /Missing abbreviation for/});
14-
assert.throws(() => checkDelegates(uppercaseLatin), { message: /Abbreviations must be all uppercase Latin letters/});
15-
assert.throws(() => checkDelegates(twoLetter), { message: /not in allowlist. New delegate abbreviations must be three letters/});
16-
assert.throws(() => checkDelegates(threeLetter), { message: /New delegate abbreviations must be three letters/});
17-
assert.throws(() => checkDelegates(duplicate), { message: /Conflicting usage on line/});
12+
assert.throws(() => checkDelegates(lex), { message: 'Line 3: Not in lexicographic order.' }); // also validates expected line number
13+
assert.throws(() => checkDelegates(missing), { message: /Missing abbreviation for/ });
14+
assert.throws(() => checkDelegates(uppercaseLatin), { message: /Abbreviations must be all uppercase Latin letters/ });
15+
assert.throws(() => checkDelegates(twoLetter), { message: /not in allowlist. New delegate abbreviations must be three letters/ });
16+
assert.throws(() => checkDelegates(threeLetter), { message: /New delegate abbreviations must be three letters/ });
17+
assert.throws(() => checkDelegates(duplicate), { message: /Conflicting usage on line/ });
1818

1919
assert.doesNotThrow(() => checkDelegates(valid));

‎scripts/check-delegates.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export function checkDelegates(contents = fs.readFileSync('./delegates.txt', 'ut
3030
for (const line of lines) {
3131
if (line.length === 0) continue;
3232
const match = re.exec(line);
33-
const {abbr} = match.groups;
33+
const { abbr } = match.groups;
3434

3535
if (previousLine.localeCompare(line, 'en') > 0) {
3636
throw new Error(`Line ${lineNumber}: Not in lexicographic order.`);

0 commit comments

Comments
 (0)