nestjs / terminus

Terminus module for Nest framework (node.js) :robot:

Home Page:https://nestjs.com/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

NestJS Terminus has an undeclared dependency on Axios

osidenate opened this issue · comments

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.

axios-import-error

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

Fixed with 9.1.1!