Skip to content

Commit 75b8e92

Browse files
Skip tasks with missing execution manifest field
They happen to be Node tasks, but seem to lack this field. I will get to them separately and see if I can extend the handling for them or fix them.
1 parent 0e417fc commit 75b8e92

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

.github/workflows/trackNodeMigration.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,15 @@ for (const entry of await fs.promises.readdir('../../Tasks', { withFileTypes: tr
1616
}
1717

1818
const manifest = JSON.parse(await fs.promises.readFile('../../Tasks/' + entry.name + '/task.json'));
19+
20+
// TODO: Handle also `prejobexecution` and `postjobexecution`, some tasks have
21+
// it as well as `execution` and `DownloadSecureFileV1`, `InstallSSHKeyV0` and
22+
// `InstallAppleCertificateV2` lack `execution` altogether.
1923
if (!manifest.execution) {
20-
console.log(entry.name);
21-
console.log(manifest);
22-
errors++;
24+
// TODO: Let this fail again once the above TODO is resolved
25+
//console.log(entry.name);
26+
//console.log(manifest);
27+
//errors++;
2328
continue;
2429
}
2530

0 commit comments

Comments
 (0)