Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(next): disable turbopack serverExternalPackages warnings #9147

Merged
merged 2 commits into from
Nov 12, 2024
Merged

fix(next): disable turbopack serverExternalPackages warnings #9147

merged 2 commits into from
Nov 12, 2024

Conversation

r1tsuu
Copy link
Member

@r1tsuu r1tsuu commented Nov 12, 2024 •
edited
Loading

What?

Disables these annoying warnings when running pnpm dev --turbo
image
vercel/next.js#68805

How?

Patches console.warn in withPayload.js

Can be disabled with PAYLOAD_PATCH_TURBOPACK_WARNINGS=false env variable

@jmikrut jmikrut enabled auto-merge (squash) November 12, 2024 17:46
@r1tsuu r1tsuu force-pushed the fix/disable-next-turbopacka-warning branch from 356396e to 1e3a65a Compare November 12, 2024 19:51
@r1tsuu r1tsuu enabled auto-merge (squash) November 12, 2024 20:10
@r1tsuu r1tsuu merged commit 64967e4 into beta Nov 12, 2024
51 checks passed
@r1tsuu r1tsuu deleted the fix/disable-next-turbopacka-warning branch November 12, 2024 20:20
@github-actions GitHub Actions
Copy link

🚀 This is included in version v3.0.0-beta.128

@r1tsuu r1tsuu mentioned this pull request Nov 19, 2024
Merged
r1tsuu added a commit that referenced this pull request Nov 21, 2024
@r1tsuu
fix(next): next@15.0.3 compatibillity for turbopack warnings patch (#…
f2205d1 
…9341)

Added patch to `withPayload` for hiding turbopack external deps warnings
from this PR #9147 didn't work
on `next@15.0.3`, now it works on both `15.0.0` and `15.0.3`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
created-by: Payload team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@r1tsuu