Skip to content

chore: add static builds to ci to ensure static builds are still working - #5617

Merged
ovflowd merged 3 commits into
mainfrom
chore/add-static-builds-to-ci
Aug 10, 2023
Merged

chore: add static builds to ci to ensure static builds are still working#5617
ovflowd merged 3 commits into
mainfrom
chore/add-static-builds-to-ci

Conversation

@ovflowd

Copy link
Copy Markdown
Member

Description

This PR simply adds static builds in addition of our regular minimal buids, to ensure that all pages are building correctly at build time and that static builds still work regardless of the changes we're doing

Validation

The current PR checks should pass

@vercel

vercelBot commented Aug 9, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
nodejs-org✅ Ready (Inspect)Visit Preview💬 Add feedbackAug 10, 2023 8:20pm

@github-actions

github-actionsBot commented Aug 9, 2023

Copy link
Copy Markdown
Contributor

Unit Test Coverage Report

LinesStatementsBranchesFunctions
Coverage: 93%
92.21% (367/398)79% (79/100)88.09% (74/84)

Unit Test Report

TestsSkippedFailuresErrorsTime
450 💤0 ❌0 🔥8.347s ⏱️

@ovflowd

Copy link
Copy Markdown
MemberAuthor

Failing on Windows due to: vercel/next.js#51225

@github-actions

Copy link
Copy Markdown
Contributor

📦 Next.js Bundle Analysis for nodejs.org

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@ovflowd
ovflowd added this pull request to the merge queueAug 10, 2023
Merged via the queue into main with commit 1fbd908Aug 10, 2023
@ovflowd
ovflowd deleted the chore/add-static-builds-to-ci branch August 10, 2023 20:37
kodiakhqBot pushed a commit to vercel/next.js that referenced this pull request Aug 10, 2023
This normalizes the backslash for windows users who accidentally use `\` in the request paths instead of the expected `/`.
For example, in nodejs/nodejs.org#5617 the slash was hardcoded for split that caused the backlash to remain the the `getStaticPaths()` result.
- Fixes#51225
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@ovflowd@bmuenzenmeyer@UlisesGascon@aymen94@araujogui