-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Closed
Description
Hi, thanks for the great work on this lib.
The removal of bluebird introduce a breaking change. I think the version number of the lib should have changed accordingly.
Indeed, the spread
method disappeared.
workbook.xlsx.readFile(file)
.then(() => {
return Promise.all([
asyncOpA(),
asyncOpB()
])
})
.spread((resA, resB) => {
//...
);
TypeError: [...].spread is not a function
I think the use of spread
is very common for bluebird users. I my case, my build was broken by this minor version change.
Ideally, we should have the possibility to inject bluebird in ExcelJS to continue to use it.
robinbullocks4rb, S4RBPhelps, kenkesey and bdsoha
Metadata
Metadata
Assignees
Labels
No labels