NestJS Terminus has an undeclared dependency on Axios
osidenate opened this issue · comments
Nathan Nelson commented
Did you read the migration guide?
- I have read the whole migration guide
Is there an existing issue that is already proposing this?
- I have searched the existing issues
Potential Commit/PR that introduced the regression
No response
Versions
8.0.6 -> 9.1.0
Describe the regression
There is a new dependency on axios
from a type definition file.
The dependency is not specified in the package.json. Including @nestjs/axios
as a peerDependency may help resolve this.
Minimum reproduction code
Expected behavior
Should be able to reference @nestjs/terminus
without having to install axios directly.
Other
No response
Livio Brunner commented
Fixed with 9.1.1!