chore(deps): remove unused dev dependencies

[micalevisk]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other... Please describe:

What is the current behavior?

Issue Number: closes #13432

What is the new behavior?

upon running npm i we'll see the following error:

npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR! 
npm ERR! While resolving: @nestjs/graphql@12.1.1
npm ERR! Found: ts-morph@22.0.0
npm ERR! node_modules/ts-morph
npm ERR!   dev ts-morph@"22.0.0" from the root project
npm ERR! 
npm ERR! Could not resolve dependency:
npm ERR! peerOptional ts-morph@"^16.0.0 || ^17.0.0 || ^18.0.0 || ^19.0.0 || ^20.0.0 || ^21.0.0" from @nestjs/graphql@12.1.1
npm ERR! node_modules/@nestjs/graphql
npm ERR!   dev @nestjs/graphql@"12.1.1" from the root project
npm ERR!   peer @nestjs/graphql@"^12.0.0" from @nestjs/apollo@12.1.0
npm ERR!   node_modules/@nestjs/apollo
npm ERR!     dev @nestjs/apollo@"12.1.0" from the root project
npm ERR! 
npm ERR! Conflicting peer dependency: ts-morph@21.0.1
npm ERR! node_modules/ts-morph
npm ERR!   peerOptional ts-morph@"^16.0.0 || ^17.0.0 || ^18.0.0 || ^19.0.0 || ^20.0.0 || ^21.0.0" from @nestjs/graphql@12.1.1
npm ERR!   node_modules/@nestjs/graphql
npm ERR!     dev @nestjs/graphql@"12.1.1" from the root project
npm ERR!     peer @nestjs/graphql@"^12.0.0" from @nestjs/apollo@12.1.0
npm ERR!     node_modules/@nestjs/apollo
npm ERR!       dev @nestjs/apollo@"12.1.0" from the root project
npm ERR! 
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR! 
npm ERR! 
npm ERR! For a full report see:
npm ERR! /home/micael/.npm/_logs/2024-04-16T13_21_01_466Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: /home/micael/.npm/_logs/2024-04-16T13_21_01_466Z-debug-0.log

but that's expected because of an outdated version ts-morph on @nestjs/graphl

Does this PR introduce a breaking change?

• Yes
• No

[coveralls]

Pull Request Test Coverage Report for Build 433cda22-a4d7-4558-a797-9ca8b96cd7ed

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 92.123%

---

Totals
Change from base Build e8e3c154-2e4b-4603-8729-3ff01d3ab650:	0.0%
Covered Lines:	6736
Relevant Lines:	7312

---

💛 - Coveralls

[Tony133]

Removing the commitlint-circle package gives this error

however the `commitlint-circle' package is used here: https://github.com/nestjs/nest/blob/master/.circleci/config.yml#L134

are you sure it is right to eliminate it ? 😄

[micalevisk]

good catch

I think we should remove the commitlint-circle because of this: