Skip to content

Commit 5ce2f4d

Browse files
Skip the Common directory and limit to tasks that do use Node
Some tasks do not use Node and use only PowerShell for example, we can skip those and they are not going to be a part of the update. The Common directory are some helpers, not a real task.
1 parent 3853186 commit 5ce2f4d

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

.github/workflows/trackNodeMigration.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,15 @@ const issues = await fetchAllPages(token, 'repos/microsoft/azure-pipelines-tasks
1010
console.log('Found', issues.length, 'Node migration issues');
1111

1212
for (const entry of await fs.promises.readdir('../../Tasks', { withFileTypes: true })) {
13-
if (!entry.isDirectory()) {
13+
if (!entry.isDirectory() || entry.name === 'Common') {
1414
continue;
1515
}
1616

17-
console.log('Parsing task', entry.name, 'manifest');
1817
const manifest = JSON.parse(await fs.promises.readFile('../../Tasks/' + entry.name + '/task.json'));
19-
console.log(entry.name, 'uses', Object.keys(manifest.execution));
18+
const keys = Object.keys(manifest.execution).filter(key => key.startsWith('Node'));
19+
if (keys.length === 0) {
20+
continue;
21+
}
22+
23+
console.log(entry.name, 'uses', keys);
2024
}

0 commit comments

Comments
 (0)