ESLint: require TS return type

versatica · Nov 8, 2024 · e062161 · e062161
1 parent c03711b
commit e062161
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -145,6 +145,7 @@ const config = tsEslint.config(
 			'@typescript-eslint/consistent-type-definitions': 0,
 			// Sorry, we need many `any` usage.
 			'@typescript-eslint/no-explicit-any': 0,
+			'@typescript-eslint/explicit-function-return-type': 2,
 			'@typescript-eslint/no-unsafe-member-access': 0,
 			'@typescript-eslint/no-unsafe-assignment': 0,
 			'@typescript-eslint/no-unsafe-call': 0,

diff --git a/node/src/DirectTransport.ts b/node/src/DirectTransport.ts
@@ -217,7 +217,7 @@ export class DirectTransport<
 	/**
 	 * Send RTCP packet.
 	 */
-	sendRtcp(rtcpPacket: Buffer) {
+	sendRtcp(rtcpPacket: Buffer): void {
 		if (!Buffer.isBuffer(rtcpPacket)) {
 			throw new TypeError('rtcpPacket must be a Buffer');
 		}

diff --git a/node/src/Producer.ts b/node/src/Producer.ts
@@ -505,7 +505,7 @@ export class Producer<
 	/**
 	 * Send RTP packet (just valid for Producers created on a DirectTransport).
 	 */
-	send(rtpPacket: Buffer) {
+	send(rtpPacket: Buffer): void {
 		if (!Buffer.isBuffer(rtpPacket)) {
 			throw new TypeError('rtpPacket must be a Buffer');
 		}