You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We created a problem (entirely my fault) when we switched to the new API with rendering strings as an option. The issue is that when you render a file rather than a string, it can be impossible for an application to decide how to handle missing dependencies.
There are, as I see it, two options:
a) check that the engine exists before reading the file to pass to the render function, this would continue to allow us to throw a synchronous error.
b) catch the error, and pass it to the callback function
In addition to this, I wonder if you'd consider extending the api to include a dependencies array so that libraries using consolidate could (if they chose to) automatically install the needed dependencies.
The text was updated successfully, but these errors were encountered:
We created a problem (entirely my fault) when we switched to the new API with rendering strings as an option. The issue is that when you render a file rather than a string, it can be impossible for an application to decide how to handle missing dependencies.
There are, as I see it, two options:
a) check that the engine exists before reading the file to pass to the render function, this would continue to allow us to throw a synchronous error.
b) catch the error, and pass it to the callback function
In addition to this, I wonder if you'd consider extending the api to include a dependencies array so that libraries using consolidate could (if they chose to) automatically install the needed dependencies.
The text was updated successfully, but these errors were encountered: