-
-
Notifications
You must be signed in to change notification settings - Fork 7.3k
Labels
feat: ssrp2-nice-to-haveNot breaking anything but nice to have (priority)Not breaking anything but nice to have (priority)performancePerformance related enhancementPerformance related enhancement
Description
Description
We currently do the isBuiltin
check on the server side. But isBuiltin
is serializable, so we can move this check to the module runner side.
isBuiltin(environment.config.resolve.builtins, url) |
That would reduce the communication between the module runner and the server.
Suggested solution
Move the check on module runner side.
Alternative
No response
Additional context
No response
Validations
- Follow our Code of Conduct
- Read the Contributing Guidelines.
- Read the docs.
- Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
Metadata
Metadata
Assignees
Labels
feat: ssrp2-nice-to-haveNot breaking anything but nice to have (priority)Not breaking anything but nice to have (priority)performancePerformance related enhancementPerformance related enhancement