Skip to content

Commit 7948e45

Browse files
Merge pull request #2478 from intuit/mandrew/private
private packages should be included in changelogs
2 parents be95283 + ff07eb5 commit 7948e45

File tree

2 files changed

+2
-16
lines changed

2 files changed

+2
-16
lines changed

packages/core/src/utils/__tests__/get-lerna-packages.test.ts

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,3 @@ test("it shouldn't included version-less packages", async () => {
2020
},
2121
]);
2222
});
23-
24-
test("it shouldn't included private packages", async () => {
25-
exec.mockReturnValue(endent`
26-
/dir/a:a:0.3.0--canary.32.d54a0c4.0
27-
/dir/b:b:0.3.0:PRIVATE
28-
`);
29-
expect(await getLernaPackages()).toStrictEqual([
30-
{
31-
name: "a",
32-
path: "/dir/a",
33-
version: "0.3.0--canary.32.d54a0c4.0",
34-
},
35-
]);
36-
});

packages/core/src/utils/get-lerna-packages.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,9 @@ export default async function getLernaPackages() {
1515
const response = await execPromise("npx", ["lerna", "ls", "-pla"]);
1616

1717
response.split("\n").forEach((packageInfo) => {
18-
const [packagePath, name, version, isPrivate] = packageInfo.split(":");
18+
const [packagePath, name, version] = packageInfo.split(":");
1919

20-
if (version !== "MISSING" && isPrivate !== "PRIVATE") {
20+
if (version !== "MISSING") {
2121
packages.push({ path: packagePath, name, version });
2222
}
2323
});

0 commit comments

Comments
 (0)