Skip to content

Conversation

@qvalentin
Copy link

@qvalentin qvalentin commented Aug 2, 2025

Prisma 6.0.0 was a breaking change but had no relevant changes for this project.

Using an up to date version of prisma is also required for deploying spliit with Nix: NixOS/nixpkgs#394621

next.config.mjs Outdated
Comment on lines 36 to 38
typescript: {
ignoreBuildErrors: true,
},
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you add this? I don't think we'd want to ignore this

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, I'm not @qvalentin or have technical insights in this pr, but I look forward to use spliit with nix.
he wrote something here and here.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last time I checked there were a bunch of ts errors in the project. Adding the typescript config to the next config was indeed required to get it to build with nix. I don't know why is is not needed when you are just using npm but it just reflects the current state of the project: ts is not used strictly.

If you want I can remove this from the PR, but I would prefer setting it since it is required for NixOS/nixpkgs#394621

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Petersmit27 I removed the change from this PR. Any chance to get the prisma update merged?

@Petersmit27
Copy link
Collaborator

Hey there, I updated Prisma to the current latest version (6.18.0) while I'm at it. Hope that also works for you :)

I'll continue the ignoreBuildErrors convo in #437

@Petersmit27 Petersmit27 closed this Nov 3, 2025
@qvalentin
Copy link
Author

Thanks, and thanks for taking care of the project :)

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