lib,src,test: lint codebase according new rules for v18.x

Signed-off-by: Juan José Arboleda <[email protected]> PR-URL: #48697 Reviewed-By: Ruy Adorno <[email protected]> Reviewed-By: Danielle Adams <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>
nodejs · Jul 9, 2023 · fcf3781 · fcf3781
1 parent 2262653
commit fcf3781
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 14 deletions.
diff --git a/lib/internal/fs/promises.js b/lib/internal/fs/promises.js
@@ -110,14 +110,6 @@ const {
   JSTransferable, kDeserialize, kTransfer, kTransferList,
 } = require('internal/worker/js_transferable');
 
-const {
-  newReadableStreamFromStreamBase,
-} = require('internal/webstreams/adapters');
-
-const {
-  readableStreamCancel,
-} = require('internal/webstreams/readablestream');
-
 const getDirectoryEntriesPromise = promisify(getDirents);
 const validateRmOptionsPromise = promisify(validateRmOptions);
 

diff --git a/lib/internal/modules/esm/loader.js b/lib/internal/modules/esm/loader.js
@@ -549,7 +549,6 @@ class ESMLoader {
    * until it reaches the bottom or short-circuits.
    * @param {URL['href']} url The URL/path of the module to be loaded
    * @param {object} context Metadata about the module
-   *
    * @returns {{ format: ModuleFormat, source: ModuleSource }}
    */
   async load(url, context = {}) {

diff --git a/pyproject.toml b/pyproject.toml
@@ -28,6 +28,7 @@ ignore = [
   "PLC1901",
   "RUF005",
   "RUF100",
+  "RUF012",
 ]
 line-length = 172
 target-version = "py37"

diff --git a/src/node_builtins.cc b/src/node_builtins.cc
@@ -365,9 +365,10 @@ MaybeLocal<Function> BuiltinLoader::LookupAndCompileInternal(
   return scope.Escape(fun);
 }
 
-MaybeLocal<Function> BuiltinLoader::LookupAndCompile(Local<Context> context,
-                                                     const char* id,
-                                                     Environment* optional_env) {
+MaybeLocal<Function> BuiltinLoader::LookupAndCompile(
+    Local<Context> context,
+    const char* id,
+    Environment* optional_env) {
   Result result;
   std::vector<Local<String>> parameters;
   Isolate* isolate = context->GetIsolate();

diff --git a/test/common/wpt.js b/test/common/wpt.js
@@ -715,7 +715,6 @@ class WPTRunner {
   /**
    * Report the status of each specific test case (there could be multiple
    * in one test file).
-   *
    * @param {string} filename
    * @param {Test} test  The Test object returned by WPT harness
    */
@@ -735,7 +734,6 @@ class WPTRunner {
 
   /**
    * Report the status of each WPT test (one per file)
-   *
    * @param {string} filename
    * @param {object} harnessStatus - The status object returned by WPT harness.
    */