ntegral / nestjs-sentry

Provides an injectable sentry.io client to provide enterprise logging nestjs modules
ISC License
380 stars 98 forks source link

Allow optionalDependencies for nestjs/graphql to 10.x.x #83

Open christopheblin opened 2 years ago

christopheblin commented 2 years ago

Replace

  "optionalDependencies": {
    "@nestjs/graphql": "^8.0.2 || ^9.0.4"
  }

by

  "optionalDependencies": {
    "@nestjs/graphql": "^8.0.2 || ^9.0.4 || ^10.0.11"
  }

to allow user of nestjs/graphql to upgrade to 10.0.11 without having a duplicate depency on nestjs/graphql 9.1.2 (which is the latest version compatible with the current range)

F3n67u commented 2 years ago

optionalDependencies already update to allow @nestjs/graphql

https://github.com/ntegral/nestjs-sentry/blob/f80c538bb827450bcd5bfb07e22b1d35ee8a12d9/package.json#LL70-L72C4

andrei2424 commented 1 year ago

Please, update @nestjs/graphql to >= 11.0.0, there are conflicts with other packages

ChopperLee2011 commented 3 months ago

@nestjs/graphql is 12.2.0 now, could @ntegral/nestjs-sentry be optimistically install the latest version like

  "optionalDependencies": {
    "@nestjs/graphql": "^12.0.11"
  }

to fix some existing and potential vulnerability issue