[eslint] function-paren-newline: error, multiline

Author: wojpawlik

.eslintrc.json

@@ -153,6 +153,7 @@
 				"allowArrowFunctions": true
 			}
 		],
+		"function-paren-newline": ["error", "multiline"],
 		"key-spacing": "error",
 		"keyword-spacing": "error",
 		"lines-around-comment": "error",

handlers/commands/addCommand.js

@@ -11,13 +11,17 @@ const addCommandHandler = async ({ chat, reply, state }) => {
 	if (chat.type !== 'private') return null;
 
 	if (!isAdmin) {
-		return reply('ℹ️ <b>Sorry, only admins access this command.</b>',
-			replyOptions);
+		return reply(
+			'ℹ️ <b>Sorry, only admins access this command.</b>',
+			replyOptions
+		);
 	}
 	await addCommand({ id: user.id });
-	return reply('Enter a name for the command without forward slash "/".' +
+	return reply(
+		'Enter a name for the command without forward slash "/".' +
 		'\n\nFor example: <b>rules</b>',
-	replyOptions);
+		replyOptions
+	);
 };
 
 module.exports = addCommandHandler;

handlers/commands/admin.js

@@ -31,8 +31,10 @@ const adminHandler = async ({ message, reply, state }) => {
 	}
 
 	if (await isAdmin(userToAdmin)) {
-		return reply(`⭐️ ${link(userToAdmin)} <b>is already admin.</b>`,
-			replyOptions);
+		return reply(
+			`⭐️ ${link(userToAdmin)} <b>is already admin.</b>`,
+			replyOptions
+		);
 	}
 
 	if (await getWarns(userToAdmin)) {

handlers/commands/ban.js

@@ -23,8 +23,10 @@ const banHandler = async ({ chat, message, reply, telegram, me, state }) => {
 	const reason = message.text.split(' ').slice(1).join(' ').trim();
 
 	if (!userToBan) {
-		return reply('ℹ️ <b>Reply to a message or mention a user.</b>',
-			replyOptions).then(scheduleDeletion);
+		return reply(
+			'ℹ️ <b>Reply to a message or mention a user.</b>',
+			replyOptions
+		).then(scheduleDeletion);
 	}
 
 	if (message.chat.type === 'private' || userToBan.username === me) {
@@ -43,12 +45,15 @@ const banHandler = async ({ chat, message, reply, telegram, me, state }) => {
 	if (message.reply_to_message) {
 		bot.telegram.deleteMessage(
 			chat.id,
-			message.reply_to_message.message_id);
+			message.reply_to_message.message_id
+		);
 	}
 
 	if (await isBanned(userToBan)) {
-		return reply(`🚫 ${link(userToBan)} <b>is already banned.</b>`,
-			replyOptions);
+		return reply(
+			`🚫 ${link(userToBan)} <b>is already banned.</b>`,
+			replyOptions
+		);
 	}
 
 	try {

handlers/commands/getwarns.js

@@ -19,8 +19,10 @@ const getWarnsHandler = async ({ message, reply, state }) => {
 			? message.commandMention
 			: null;
 	if (!theUser) {
-		return reply('ℹ️ <b>Reply to a message or mention a user.</b>',
-			replyOptions).then(scheduleDeletion);
+		return reply(
+			'ℹ️ <b>Reply to a message or mention a user.</b>',
+			replyOptions
+		).then(scheduleDeletion);
 	}
 	let i = 0;
 	const warns = await getWarns(theUser);

handlers/commands/groups.js

@@ -25,11 +25,10 @@ const groupsHandler = async ({ replyWithHTML }) => {
 
 	const entries = groups.map(entry).join('\n');
 
-	return replyWithHTML(
-		`🛠 <b>Groups I manage</b>:\n\n${entries}`, {
-			disable_web_page_preview: true,
-			reply_markup,
-		}).then(scheduleDeletion);
+	return replyWithHTML(`🛠 <b>Groups I manage</b>:\n\n${entries}`, {
+		disable_web_page_preview: true,
+		reply_markup,
+	}).then(scheduleDeletion);
 };
 
 module.exports = groupsHandler;

handlers/commands/help.js

@@ -23,9 +23,10 @@ const helpHandler = ({ chat, replyWithHTML }) => {
 
 	return replyWithHTML(
 		message,
-		Markup.inlineKeyboard(
-			[ Markup.urlButton('🛠 Setup a New Bot', homepage) ])
-			.extra());
+		Markup.inlineKeyboard([
+			Markup.urlButton('🛠 Setup a New Bot', homepage)
+		]).extra()
+	);
 };
 
 module.exports = helpHandler;

handlers/commands/leave.js

@@ -15,17 +15,17 @@ const leaveCommandHandler = async ctx => {
 			:	{ title: groupName };
 		const isGroup = await managesGroup(group);
 		if (!isGroup) {
+			// eslint-disable-next-line function-paren-newline
 			return replyWithHTML(
 				'ℹ️ <b>Couldn\'t find a group with that ID/name.</b>'
+			// eslint-disable-next-line function-paren-newline
 			);
 		}
 		await Promise.all([
 			removeGroup(isGroup),
 			telegram.leaveChat(isGroup.id)
 		]);
-		return replyWithHTML(
-			'✅ <b>I no longer manage that group.</b>'
-		);
+		return replyWithHTML('✅ <b>I no longer manage that group.</b>');
 	}
 
 	await removeGroup(chat);

handlers/commands/link.js

@@ -9,10 +9,8 @@ const { managesGroup } = require('../../stores/group');
 const linkHandler = async ({ chat, replyWithHTML }) => {
 	const group = await managesGroup({ id: chat.id });
 
-	return replyWithHTML(
-		'ℹ️ <b>Group\'s link:</b>\n\n' +
-		`<a href="${group.link}">${group.title}</a>`
-	).then(scheduleDeletion);
+	return replyWithHTML('ℹ️ <b>Group\'s link:</b>\n\n' +
+		`<a href="${group.link}">${group.title}</a>`).then(scheduleDeletion);
 };
 
 module.exports = linkHandler;

handlers/commands/nowarns.js

@@ -21,15 +21,19 @@ const nowarnsHandler = async ({ message, reply, state }) => {
 			: null;
 
 	if (!userToUnwarn) {
-		return reply('ℹ️ <b>Reply to a message or mention a user.</b>',
-			replyOptions).then(scheduleDeletion);
+		return reply(
+			'ℹ️ <b>Reply to a message or mention a user.</b>',
+			replyOptions
+		).then(scheduleDeletion);
 	}
 
 	const warns = await getWarns(userToUnwarn);
 
 	if (!warns) {
-		return reply(`ℹ️ ${link(userToUnwarn)} <b>already has no warnings.</b>`,
-			replyOptions);
+		return reply(
+			`ℹ️ ${link(userToUnwarn)} <b>already has no warnings.</b>`,
+			replyOptions
+		);
 	}
 	try {
 		await nowarns(userToUnwarn);
@@ -40,7 +44,8 @@ const nowarnsHandler = async ({ message, reply, state }) => {
 	return reply(
 		`♻️ ${link(user)} <b>pardoned</b> ${link(userToUnwarn)} ` +
 		'<b>for all of their warnings.</b>',
-		replyOptions);
+		replyOptions
+	);
 };