Skip to content

Commit 23f8a25

Browse files
authoredSep 9, 2024··
fix: use cp and rmdir for copy_assets action (#1329)
1 parent ed64bee commit 23f8a25

File tree

2 files changed

+14
-2
lines changed

2 files changed

+14
-2
lines changed
 

‎.changeset/violet-clocks-fetch.md

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'style-dictionary': patch
3+
---
4+
5+
Use cp and rmdir commands for `copy_assets` do and undo methods, since they affect directories, not files.

‎lib/common/actions.js

+9-2
Original file line numberDiff line numberDiff line change
@@ -78,14 +78,21 @@ export default {
7878
// eslint-disable-next-line no-console
7979
console.log('Copying assets directory to ' + config.buildPath + 'assets');
8080
}
81-
return vol.promises.copyFile('assets', config.buildPath + 'assets');
81+
return vol.promises.cp(
82+
'assets',
83+
config.buildPath + 'assets',
84+
// @ts-expect-error ICpOptions requires other props, this is a mistake in memfs types definition
85+
{
86+
recursive: true,
87+
},
88+
);
8289
},
8390
undo: async function (_, config, options, vol = fs) {
8491
if (config.log?.verbosity !== 'silent') {
8592
// eslint-disable-next-line no-console
8693
console.log('Removing assets directory from ' + config.buildPath + 'assets');
8794
}
88-
return vol.promises.unlink(config.buildPath + 'assets');
95+
return vol.promises.rmdir(config.buildPath + 'assets', { recursive: true });
8996
},
9097
},
9198
};

0 commit comments

Comments
 (0)
Please sign in to comment.