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.

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

Use Object.assign instead of object spread for serverless loader #6056

Merged
merged 1 commit into from
Jan 15, 2019

Conversation

timneutkens
Copy link
Member

Fixes https://github.com/zeit/now-builders/issues/168

For some reason with a certain mix of deps ... is not supported in webpack's parsing.
By default it is supported as all our tests passed before and we have deployed Next.js apps on v2 already.

@timneutkens timneutkens merged commit da3c3ad into vercel:canary Jan 15, 2019
@timneutkens timneutkens deleted the fix/serverless-target-parse branch January 15, 2019 01:20
timneutkens added a commit to timneutkens/next.js that referenced this pull request Jan 17, 2019
…cel#6056)

Fixes vercel/now-builders#168

For some reason with a certain mix of deps `...` is not supported in webpack's parsing.
By default it is supported as all our tests passed before and we have deployed Next.js apps on v2 already.
@lock lock bot locked as resolved and limited conversation to collaborators Jan 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants