Skip to content

Commit

Permalink
tools: make mailmap processing for author list case-insensitive
Browse files Browse the repository at this point in the history
This is to accommodate Myles Borins otherwise ending up with
multiple entries due to different casing in the email 🙂

PR-URL: #29608
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Saúl Ibarra Corretgé <saghul@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
addaleax authored and targos committed Oct 1, 2019
1 parent a86b71f commit c361180
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion tools/update-authors.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,13 @@ const path = require('path');
const fs = require('fs');
const readline = require('readline');

class CaseIndifferentMap {
_map = new Map();

get(key) { return this._map.get(key.toLowerCase()); }
set(key, value) { return this._map.set(key.toLowerCase(), value); }
}

const log = spawn(
'git',
// Inspect author name/email and body.
Expand All @@ -23,7 +30,7 @@ else

output.write('# Authors ordered by first contribution.\n\n');

const mailmap = new Map();
const mailmap = new CaseIndifferentMap();
{
const lines = fs.readFileSync(path.resolve(__dirname, '../', '.mailmap'),
{ encoding: 'utf8' }).split('\n');
Expand Down

0 comments on commit c361180

Please sign in to comment.