tools: fix formatting of warning message in update-authors.js · nodejs/node@8648e50 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 635
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 8648e50

Browse filesBrowse the repository at this point in the historyBrowse files

Trott

authored and

BethGriggs

committed

tools: fix formatting of warning message in update-authors.js

The email already has `<` and `>` surrounding it so adding it results in a misformatted message. PR-URL: #40600 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Voltrex <mohammadkeyvanzade94@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 parent e2114e2 commit 8648e50
Copy full SHA for 8648e50

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• tools
  • update-authors.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎tools/update-authors.js‎

Copy file name to clipboardExpand all lines: tools/update-authors.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -121,7 +121,7 @@ rl.on('line', (line) => {
121	121	const duplicate = previousAuthors.get(author);
122	122	if (duplicate && !duplicate.includes(email)) {
123	123	console.warn('Author name already in AUTHORS file. Possible duplicate:');
124		- console.warn(` ${author} <${email}>`);
	124	+ console.warn(` ${author} ${email}`);
125	125	}
126	126	});
127	127

0 commit comments

Comments

0 (0)