Make pnpm optional for setup flow

This commit is contained in:
Will Chen
2025-04-21 15:01:34 -07:00
parent 187aab7682
commit 50de3baa10

View File

@@ -83,9 +83,8 @@ export function SetupBanner() {
await checkNode(); await checkNode();
}, [checkNode, setNodeInstallStep]); }, [checkNode, setNodeInstallStep]);
const isNodeSetupComplete = Boolean( // We only check for node version because pnpm is not required for the app to run.
nodeSystemInfo?.nodeVersion && nodeSystemInfo?.pnpmVersion const isNodeSetupComplete = Boolean(nodeSystemInfo?.nodeVersion);
);
const itemsNeedAction: string[] = []; const itemsNeedAction: string[] = [];
if (!isNodeSetupComplete && nodeSystemInfo) { if (!isNodeSetupComplete && nodeSystemInfo) {
@@ -161,7 +160,8 @@ export function SetupBanner() {
Node.js ({nodeSystemInfo!.nodeVersion}) installed.{" "} Node.js ({nodeSystemInfo!.nodeVersion}) installed.{" "}
{nodeSystemInfo!.pnpmVersion && ( {nodeSystemInfo!.pnpmVersion && (
<span className="text-xs text-gray-500"> <span className="text-xs text-gray-500">
pnpm ({nodeSystemInfo!.pnpmVersion}) installed. {" "}
(optional) pnpm ({nodeSystemInfo!.pnpmVersion}) installed.
</span> </span>
)} )}
</p> </p>