Skip to content

Commit 9e25197

Browse files
authored
Merge pull request #16 from simple-frontend-dev/fix-ts
dev: make it obvious for browser TS setup to follow vite templates
2 parents 7ae8886 + ce695a4 commit 9e25197

2 files changed

Lines changed: 9 additions & 1 deletion

File tree

.changeset/ten-chefs-warn.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"simplefrontend": patch
3+
---
4+
5+
improve ts setup success message for browser env

src/solutions/typescript.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,9 @@ export async function setupTypescript() {
4949
return;
5050
} else if (openViteStarter) {
5151
await open("https://vite.dev/guide/#trying-vite-online");
52+
log.info(
53+
"To finish your TypeScript setup, I recommend you follow the recommended configurations from Vite starters at https://vite.dev/guide/#trying-vite-online ",
54+
);
5255
}
5356
} else if (environment === "node") {
5457
// step 4: select the build context
@@ -97,8 +100,8 @@ export async function setupTypescript() {
97100
JSON.stringify(typeScriptConfigurationServer, null, 2),
98101
);
99102
}
103+
log.success("Successfully setup TypeScript and created tsconfig.json");
100104
}
101-
log.success("Successfully setup TypeScript and created tsconfig.json");
102105
} catch (error: unknown) {
103106
log.error(`Failed to install typescript - error: ${error}`);
104107
}

0 commit comments

Comments
 (0)