Skip to content

Commit 2afbe91

Browse files
authored
fix: ensure glob finds all files in folders (#10230)
For some reason {folders.value.join(',')} as part of the glob doesn't work and returns less files fixes #10228
1 parent 96c9237 commit 2afbe91

File tree

2 files changed

+12
-3
lines changed

2 files changed

+12
-3
lines changed

.changeset/flat-berries-stare.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
'svelte-migrate': patch
3+
---
4+
5+
fix: ensure glob finds all files in folders

packages/migrate/migrations/svelte-4/index.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,13 @@ export async function migrate() {
6767
'.svelte'
6868
];
6969
const extensions = [...svelte_extensions, '.ts', '.js'];
70-
const files = glob(`{${folders.value.join(',')}}/**`, { filesOnly: true, dot: true })
71-
.map((file) => file.replace(/\\/g, '/'))
72-
.filter((file) => !file.includes('/node_modules/'));
70+
// For some reason {folders.value.join(',')} as part of the glob doesn't work and returns less files
71+
const files = folders.value.flatMap(
72+
/** @param {string} folder */ (folder) =>
73+
glob(`${folder}/**`, { filesOnly: true, dot: true })
74+
.map((file) => file.replace(/\\/g, '/'))
75+
.filter((file) => !file.includes('/node_modules/'))
76+
);
7377

7478
for (const file of files) {
7579
if (extensions.some((ext) => file.endsWith(ext))) {

0 commit comments

Comments
 (0)