Skip to content

Commit 9be2899

Browse files
committed
Ignore external packages in action sorter
1 parent fba1b26 commit 9be2899

2 files changed

Lines changed: 20 additions & 1 deletion

File tree

.github/actions/src/info/__tests__/sorter.test.ts

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,25 @@ describe(topoSortPackages, () => {
4747
]);
4848
});
4949

50+
test('Ignores external Source Academy dependencies', () => {
51+
const result = topoSortPackages({
52+
'@sourceacademy/0': {
53+
type: null,
54+
hasChanges: false,
55+
directory: '/',
56+
package: {
57+
name: '@sourceacademy/0',
58+
devDependencies: {},
59+
dependencies: {
60+
'@sourceacademy/external': '^1.0.0'
61+
}
62+
}
63+
}
64+
});
65+
66+
expect(result).toEqual(['@sourceacademy/0']);
67+
});
68+
5069
test('With a cycle', () => {
5170
const func = () => topoSortPackages({
5271
'@sourceacademy/0': {

.github/actions/src/info/sorter.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ export function topoSortPackages(packages: Record<string, RawPackageRecord>) {
2828
}
2929

3030
for (const name of keys) {
31-
if (name.startsWith('@sourceacademy')) {
31+
if (name in packages) {
3232
if (!(name in indegrees)) {
3333
indegrees[name] = 1;
3434
} else {

0 commit comments

Comments
 (0)