Commit f22f622
lib: add lint rules for reflective function calls
PR-URL: #60825
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jordan Harband <ljharb@gmail.com>
Reviewed-By: LiviaMedeiros <livia@cirno.name>1 parent dcffa88 commit f22f622
4 files changed
Lines changed: 10 additions & 16 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1118,11 +1118,7 @@ function lazyLoadRimraf() { | |||
| 1118 | 1118 | /** | |
| 1119 | 1119 | * Asynchronously removes a directory. | |
| 1120 | 1120 | * @param {string | Buffer | URL} path | |
| 1121 | - * @param {{ | ||
| 1122 | - * maxRetries?: number; | ||
| 1123 | - * recursive?: boolean; | ||
| 1124 | - * retryDelay?: number; | ||
| 1125 | - * }} [options] | ||
| 1121 | + * @param {object} [options] | ||
| 1126 | 1122 | * @param {(err?: Error) => any} callback | |
| 1127 | 1123 | * @returns {void} | |
| 1128 | 1124 | */ | |
@@ -1166,11 +1162,7 @@ function rmdir(path, options, callback) { | |||
| 1166 | 1162 | /** | |
| 1167 | 1163 | * Synchronously removes a directory. | |
| 1168 | 1164 | * @param {string | Buffer | URL} path | |
| 1169 | - * @param {{ | ||
| 1170 | - * maxRetries?: number; | ||
| 1171 | - * recursive?: boolean; | ||
| 1172 | - * retryDelay?: number; | ||
| 1173 | - * }} [options] | ||
| 1165 | + * @param {object} [options] | ||
| 1174 | 1166 | * @returns {void} | |
| 1175 | 1167 | */ | |
| 1176 | 1168 | function rmdirSync(path, options) { | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -896,7 +896,7 @@ class QuicSession { | |||
| 896 | 896 | #onstream = undefined; | |
| 897 | 897 | /** @type {OnDatagramCallback|undefined} */ | |
| 898 | 898 | #ondatagram = undefined; | |
| 899 | - /** @type {{}} */ | ||
| 899 | + /** @type {object|undefined} */ | ||
| 900 | 900 | #sessionticket = undefined; | |
| 901 | 901 | ||
| 902 | 902 | /** | |
@@ -927,7 +927,10 @@ class QuicSession { | |||
| 927 | 927 | return this.#handle === undefined || this.#isPendingClose; | |
| 928 | 928 | } | |
| 929 | 929 | ||
| 930 | - /** @type {any} */ | ||
| 930 | + /** | ||
| 931 | + * Get the session ticket associated with this session, if any. | ||
| 932 | + * @type {object|undefined} | ||
| 933 | + */ | ||
| 931 | 934 | get sessionticket() { return this.#sessionticket; } | |
| 932 | 935 | ||
| 933 | 936 | /** @type {OnStreamCallback} */ | |
@@ -1665,7 +1668,7 @@ class QuicEndpoint { | |||
| 1665 | 1668 | ||
| 1666 | 1669 | /** | |
| 1667 | 1670 | * Initiates a session with a remote endpoint. | |
| 1668 | - * @param {{}} address | ||
| 1671 | + * @param {object} address | ||
| 1669 | 1672 | * @param {SessionOptions} [options] | |
| 1670 | 1673 | * @returns {QuicSession} | |
| 1671 | 1674 | */ | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -867,8 +867,7 @@ function newStreamDuplexFromReadableWritablePair(pair = kEmptyObject, options = | |||
| 867 | 867 | ||
| 868 | 868 | /** | |
| 869 | 869 | * @typedef {import('./queuingstrategies').QueuingStrategy} QueuingStrategy | |
| 870 | - * @typedef {{}} StreamBase | ||
| 871 | - * @param {StreamBase} streamBase | ||
| 870 | + * @param {object} streamBase | ||
| 872 | 871 | * @param {QueuingStrategy} strategy | |
| 873 | 872 | * @returns {WritableStream} | |
| 874 | 873 | */ | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -327,7 +327,7 @@ function getHeapSnapshotOptionTests() { | |||
| 327 | 327 | } | |
| 328 | 328 | ||
| 329 | 329 | /** | |
| 330 | - * Similar to @see {validateByRetainingPathFromNodes} but creates the snapshot from scratch. | ||
| 330 | + * Similar to {@link validateByRetainingPathFromNodes} but creates the snapshot from scratch. | ||
| 331 | 331 | * @returns {object[]} | |
| 332 | 332 | */ | |
| 333 | 333 | function validateByRetainingPath(...args) { | |
0 commit comments