Skip to content

[WIP] Chore(deps): update vitest monorepo to ^4.1.10#1253

Draft
underfisk with Copilot wants to merge 1 commit into
renovate/vitest-monorepofrom
copilot/choredeps-update-vitest-monorepo
Draft

[WIP] Chore(deps): update vitest monorepo to ^4.1.10#1253
underfisk with Copilot wants to merge 1 commit into
renovate/vitest-monorepofrom
copilot/choredeps-update-vitest-monorepo

Conversation

Copilot AI commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Thanks for the feedback on #1252. I've created this new PR, which merges into #1252, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.

Original PR: #1252
Triggering comment (#1252 (comment)):

@copilot can you investigate the failing tests? Is it related to drift on the nestjs versions between the dev dependencies/integration tests? It's a known DI issue if Nestjs has different versions...

I dug into it, but a safe fix here is bigger than this dependency PR. The DI failure is in @golevelup/nestjs-discovery requiring Nest internals (ModulesContainer) during test module bootstrap, and the Stripe failures are downstream of that bootstrap failure.

I can put up a follow-up PR that updates discovery test/bootstrap wiring for Nest 11 (and adjusts affected Stripe tests) as a dedicated fix. If you want, I’ll proceed with that approach.

Please do, proceed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants