-
Notifications
You must be signed in to change notification settings - Fork 470
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
feat(SSR): split building index.json and SSR into two #10953
base: main
Are you sure you want to change the base?
Conversation
7091f33
to
bbcab3d
Compare
This pull request has merge conflicts that must be resolved before it can be merged. |
4e5e120
to
d6e8dd6
Compare
"build:prepare": "yarn build:client && yarn build:ssr && yarn tool popularities && yarn tool spas && yarn tool gather-git-history && yarn tool build-robots-txt", | ||
"build:render-html": "cross-env NODE_ENV=production NODE_OPTIONS='--no-warnings=ExperimentalWarning --loader ts-node/esm' node build/cli.ts render", |
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.
Curious, why you didn't call it html
for consistency with build:json
?
This pull request has merge conflicts that must be resolved before it can be merged. |
This pull request has merge conflicts that must be resolved before it can be merged. |
Summary
Work in progress for transparency.
Problem
Solution
Screenshots
Before
After
How did you test this change?