-
-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(templates): add SSG template #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Workflow looks great to me!
36bcd6c
to
08c05e8
Compare
08c05e8
to
8251d62
Compare
prompts.confirm({ | ||
message: 'Use router?', | ||
initialValue: false, | ||
}), | ||
}) : Promise.resolve(false), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rschristian do you know if there is a better way to do this? It looks like it goofs up our types
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not, no. Clack does have some wonky types in places but it's been something I'm fine with // @ts-ignore
'ing or whatever if the red squiggle is annoying for people.
We can ungroup the prompts, do something like this to get better control flow, but it's perhaps a bit repetitive.
8251d62
to
19ca1ae
Compare
* ssg template * todos * fixes * use appType * ignore error
This adds an SSG-template using
vite-plugin-ssr
, I am however curious how we'd incorporate this in our current setup would we do-> base/ssg?
--> base ? use-router? : none
--> useTS ?