chore(deps): bump @nestjs/core from 11.1.12 to 11.1.28 in /queueConsumer#2332
Conversation
Greptile SummaryThis PR bumps the NestJS core dependency used by the queue consumer.
Confidence Score: 4/5This should be fixed before merging.
queueConsumer/package.json and the pnpm lockfile
|
| Filename | Overview |
|---|---|
| queueConsumer/package.json | Bumps @nestjs/core for the queue consumer package. |
Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
queueConsumer/package.json:31
**Lockfile Still Mismatched**
This manifest now requires exact `@nestjs/core@11.1.28`, but the committed pnpm lockfile still records the queue consumer dependency with the old `11.1.12` specifier and resolution. When CI or the Docker build installs from the committed lockfile, pnpm can reject the install before the queue consumer builds or starts. Please commit the matching lockfile update with this package bump.
Reviews (2): Last reviewed commit: "chore(deps): bump @nestjs/core from 11.1..." | Re-trigger Greptile
| "@nestjs/common": "^11.1.12", | ||
| "@nestjs/config": "^4.0.4", | ||
| "@nestjs/core": "11.1.12", | ||
| "@nestjs/core": "11.1.27", |
There was a problem hiding this comment.
Frozen Install Lockfile Mismatch
When CI or deployment runs pnpm with a frozen lockfile, this manifest now requires @nestjs/core@11.1.27 but the PR does not include the matching lockfile change. That can make the queue consumer fail before build or startup instead of installing the dependency graph declared here.
Prompt To Fix With AI
This is a comment left during a code review.
Path: queueConsumer/package.json
Line: 31
Comment:
**Frozen Install Lockfile Mismatch**
When CI or deployment runs pnpm with a frozen lockfile, this manifest now requires `@nestjs/core@11.1.27` but the PR does not include the matching lockfile change. That can make the queue consumer fail before build or startup instead of installing the dependency graph declared here.
How can I resolve this? If you propose a fix, please make it concise.Bumps [@nestjs/core](https://github.com/nestjs/nest/tree/HEAD/packages/core) from 11.1.12 to 11.1.28. - [Release notes](https://github.com/nestjs/nest/releases) - [Commits](https://github.com/nestjs/nest/commits/v11.1.28/packages/core) --- updated-dependencies: - dependency-name: "@nestjs/core" dependency-version: 11.1.27 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
222c99b to
676df8a
Compare
| "@nestjs/common": "^11.1.12", | ||
| "@nestjs/config": "^4.0.4", | ||
| "@nestjs/core": "11.1.12", | ||
| "@nestjs/core": "11.1.28", |
There was a problem hiding this comment.
This manifest now requires exact @nestjs/core@11.1.28, but the committed pnpm lockfile still records the queue consumer dependency with the old 11.1.12 specifier and resolution. When CI or the Docker build installs from the committed lockfile, pnpm can reject the install before the queue consumer builds or starts. Please commit the matching lockfile update with this package bump.
Prompt To Fix With AI
This is a comment left during a code review.
Path: queueConsumer/package.json
Line: 31
Comment:
**Lockfile Still Mismatched**
This manifest now requires exact `@nestjs/core@11.1.28`, but the committed pnpm lockfile still records the queue consumer dependency with the old `11.1.12` specifier and resolution. When CI or the Docker build installs from the committed lockfile, pnpm can reject the install before the queue consumer builds or starts. Please commit the matching lockfile update with this package bump.
How can I resolve this? If you propose a fix, please make it concise.
Bumps @nestjs/core from 11.1.12 to 11.1.28.
Release notes
Sourced from @nestjs/core's releases.
... (truncated)
Commits
dfaa376chore(release): publish v11.1.28 release053a316Merge pull request #17239 from jyx-07/fix/sse-interceptor-teardown8e7a2eefeat(core): include auto-converted route in legacy route path warningc5b7164fix(core): trigger sse observable teardown on disconnect660a124chore(release): publish v11.1.27 release5c5e9bafix(core): sse async handlers teardown issue #17128aa5c4f1chore: update readme and package.json9ff83d5chore(release): publish v11.1.26 release0f398fdtest: fix broken unit testd152eecfix(core): post sse endpoint empty response #17098