-
Notifications
You must be signed in to change notification settings - Fork 340
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
fix: support multiple path values in edge functions config #5480
Conversation
|
||
if (index === -1) { | ||
declarations.push(declarationFromSource) | ||
const { path: pathOrPaths, ...configProps } = declarationFromSource |
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.
Not a fan of this. I think Edge Bundler should be the one merging the two config sources, since where most of that existing logic lives.
📊 Benchmark resultsComparing with 5011be5 Package size: 263 MB(no change)
Legend
|
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.
Looks good :) let's get this merged
Summary
Fixes an issue where an error is thrown for edge functions defining a
path
property with multiple values via in-source configuration.