Skip to content

Commit

Permalink
Merge pull request #2478 from intuit/mandrew/private
Browse files Browse the repository at this point in the history
private packages should be included in changelogs
  • Loading branch information
hipstersmoothie authored Oct 25, 2024
2 parents be95283 + ff07eb5 commit 7948e45
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 16 deletions.
14 changes: 0 additions & 14 deletions packages/core/src/utils/__tests__/get-lerna-packages.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,17 +20,3 @@ test("it shouldn't included version-less packages", async () => {
},
]);
});

test("it shouldn't included private packages", async () => {
exec.mockReturnValue(endent`
/dir/a:a:0.3.0--canary.32.d54a0c4.0
/dir/b:b:0.3.0:PRIVATE
`);
expect(await getLernaPackages()).toStrictEqual([
{
name: "a",
path: "/dir/a",
version: "0.3.0--canary.32.d54a0c4.0",
},
]);
});
4 changes: 2 additions & 2 deletions packages/core/src/utils/get-lerna-packages.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,9 @@ export default async function getLernaPackages() {
const response = await execPromise("npx", ["lerna", "ls", "-pla"]);

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

if (version !== "MISSING" && isPrivate !== "PRIVATE") {
if (version !== "MISSING") {
packages.push({ path: packagePath, name, version });
}
});
Expand Down

0 comments on commit 7948e45

Please sign in to comment.