Skip to content

fix: add esbuild devDependency required by vite 8 - #300

Merged
ntnn merged 2 commits into
mainfrom
fix-esbuild
Nov 10, 2026
Merged

ntnn merged 2 commits into
mainfrom
fix-esbuild

Conversation

@ntnn

@ntnn ntnn commented Nov 6, 2026

Copy link
Copy Markdown
Member

Vite 8 no longer bundles esbuild; VitePress 1.6.4 still calls transformWithEsbuild, which broke npm run build in CI. Adds esbuild as a devDependency.

Note: this branch is based on #297's head (which bumps the vite override to 8.3.5), so it includes those changes. Alternatively, cherry-pick bc5fde2 onto #297.

@ntnn
ntnn requested review from a team as code owners November 6, 2026 18:24
@ntnn
ntnn enabled auto-merge (rebase) November 6, 2026 18:25
Signed-off-by: Nelo-T. Wallus <git@ntnn.de>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>
@xrstf

xrstf commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Signed-off-by: Nelo-T. Wallus <git@ntnn.de>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.2
Preview removed because the pull request was closed.
2026-11-10 12:37 UTC

@ntnn

ntnn commented Nov 9, 2026

Copy link
Copy Markdown
Member Author

Run marocchino/sticky-pull-request-comment@67d0dec
Error: Resource not accessible by integration - docs.github.com/rest/issues/comments#create-an-issue-comment

fixed ✌️

@ntnn
ntnn disabled auto-merge November 10, 2026 12:36
@ntnn
ntnn merged commit 803703e into main Nov 10, 2026
7 checks passed
@ntnn
ntnn deleted the fix-esbuild branch November 10, 2026 12:36
Sign up for free to 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.

2 participants