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
To fix this error I will need to change the public API for rethinkdb-job-queue which in turn means bumping the version up to the next major version due to SemVer.
The specific Public API change will be moving from having the Queue.id as the first argument for all events to having error objects as the first argument for error events. I will add the Queue.id to the error objects before raising the event.
The text was updated successfully, but these errors were encountered:
I had a few hours up my sleeve and decided to write the last couple of tests to get the coverage up.
In doing so I discovered an API design flaw which caused an exception whilst throwing
error
events.I posted on the Nodejs Help repository and @bnoordhuis has raised a pull request for Nodejs.
To fix this error I will need to change the public API for
rethinkdb-job-queue
which in turn means bumping the version up to the next major version due to SemVer.The specific Public API change will be moving from having the
Queue.id
as the first argument for all events to havingerror
objects as the first argument for error events. I will add theQueue.id
to the error objects before raising the event.The text was updated successfully, but these errors were encountered: