fix(fs): Refactor copy and copySync functions to simplify return type

This commit is contained in:
Philipp Kunz 2025-01-29 18:10:48 +01:00
parent 0abbe8bbd7
commit 13081b7344
3 changed files with 11 additions and 13 deletions

View File

@ -1,5 +1,11 @@
# Changelog
## 2025-01-29 - 11.1.7 - fix(fs)
Refactor copy and copySync functions to simplify return type
- Changed the return type of fs.copy and fs.copySync from boolean to void.
- Removed unnecessary promise handling in fs.copy.
## 2025-01-29 - 11.1.6 - fix(fs)
Fix issues with fs file copy functions.

View File

@ -3,6 +3,6 @@
*/
export const commitinfo = {
name: '@push.rocks/smartfile',
version: '11.1.6',
version: '11.1.7',
description: 'Provides comprehensive tools for efficient file management in Node.js using TypeScript, including handling streams, virtual directories, and various file operations.'
}

View File

@ -74,23 +74,15 @@ export const isFile = (pathArg): boolean => {
/**
* copies a file or directory from A to B on the local disk
*/
export const copy = async (fromArg: string, toArg: string): Promise<boolean> => {
const done = plugins.smartpromise.defer<boolean>();
plugins.fsExtra.copy(fromArg, toArg, {}, (err) => {
if (err) {
throw new Error(`Could not copy from ${fromArg} to ${toArg}: ${err}`);
}
done.resolve(true);
});
return done.promise;
export const copy = async (fromArg: string, toArg: string): Promise<void> => {
return await plugins.fsExtra.copy(fromArg, toArg, {});
};
/**
* copies a file or directory SYNCHRONOUSLY from A to B on the local disk
*/
export const copySync = (fromArg: string, toArg: string): boolean => {
plugins.fsExtra.copySync(fromArg, toArg);
return true;
export const copySync = (fromArg: string, toArg: string): void => {
return plugins.fsExtra.copySync(fromArg, toArg);
};
/**