Skip to content

Conversation

@mag123c
Copy link
Contributor

@mag123c mag123c commented Nov 28, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

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: Sample(22) update for Prisma 7 compatibility

What is the current behavior?

Sample 22-graphql-prisma fails with Prisma 7 due to breaking changes

Issue Number: #15965

What is the new behavior?

  • Removed url from schema.prisma datasource block
  • Added prisma.config.ts for datasource configuration
  • Added moduleFormat = "cjs" for NestJS CommonJS compatibility
  • Updated prisma.service.ts to use @prisma/adapter-better-sqlite3

Does this PR introduce a breaking change?

  • Yes
  • No

@coveralls
Copy link

Pull Request Test Coverage Report for Build b9252815-71a8-4b9b-844a-abd377a3910b

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 88.939%

Totals Coverage Status
Change from base Build c89172e7-62df-46f3-b84f-5b3077430054: 0.0%
Covered Lines: 7325
Relevant Lines: 8236

💛 - Coveralls

@kamilmysliwiec kamilmysliwiec merged commit 9df4ac3 into nestjs:master Nov 28, 2025
4 checks passed
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.

3 participants