summaryrefslogtreecommitdiff
path: root/ext/node/clippy.toml
diff options
context:
space:
mode:
authorBartek IwaƄczuk <biwanczuk@gmail.com>2023-03-30 03:20:31 +0200
committerGitHub <noreply@github.com>2023-03-30 03:20:31 +0200
commit913e2875c1c31d5ffbc9c0c9ed0e8c63f6143024 (patch)
tree439fa7eef067893fde75dfe386d801a090e6c057 /ext/node/clippy.toml
parent89bbbd102c6050763105be39a49494a5ffdce35b (diff)
refactor(ext/node): add NodeEnv::Fs associated type (#18484)
This commit adds associated type to "NodeEnv" trait, called "Fs". The "Fs" type has a trait bound on "NodeFs", which specifies APIs required for all ops and resolution APIs to function. A "RealFs" implementation of "NodeFs" is exported from the "deno_node" crate, that provides a default implementation for the trait. All code in "deno_node" extension was changed to use the "NodeFs" trait to handle file system operations, instead of relying on APIs from the standard library.
Diffstat (limited to 'ext/node/clippy.toml')
-rw-r--r--ext/node/clippy.toml21
1 files changed, 21 insertions, 0 deletions
diff --git a/ext/node/clippy.toml b/ext/node/clippy.toml
new file mode 100644
index 000000000..3ce5624b3
--- /dev/null
+++ b/ext/node/clippy.toml
@@ -0,0 +1,21 @@
+disallowed-methods = [
+ { path = "std::env::current_dir", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::path::Path::exists", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::canonicalize", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::copy", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::create_dir", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::create_dir_all", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::hard_link", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::metadata", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::read", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::read_dir", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::read_link", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::read_to_string", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::remove_dir", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::remove_dir_all", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::remove_file", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::rename", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::set_permissions", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::symlink_metadata", reason = "File system operations should be done using NodeFs trait" },
+ { path = "std::fs::write", reason = "File system operations should be done using NodeFs trait" },
+]