Skip to content

Conversation

@nikita-volkov
Copy link

This is exclusively a structural refactoring of the PostgREST.Error module, which decouples various errors by distributing them across cohesive modules. It also encapsulates the helper definitions.

  • No changes to the API of the refactored module.
  • No changes in its implementation.
  • No renamings.
  • Reformatting avoided as much as possible.

I've noticed myself needing this refactoring while working on migrating the codebase to the upcoming release of Hasql. After this refactoring that concern is now clearly located under the PostgREST.Error.PgError.* namespace.

@wolfgangwalther
Copy link
Member

while working on migrating the codebase to the upcoming release of Hasql

Please note that we are still stuck on the versioning we are currently using, because we still need to support GHC 9.4 for the time being. So there is no quick way forward. Also some other PRs already have code to update hasql, once we get there, for example 612dfec.

@nikita-volkov
Copy link
Author

because we still need to support GHC 9.4 for the time being. So there is no quick way forward.

The latest releases of Hasql and the upcoming version is tested to support GHC down to 8.10. So there should be no issue with that.

@wolfgangwalther
Copy link
Member

The latest releases of Hasql and the upcoming version is tested to support GHC down to 8.10. So there should be no issue with that.

Interesting. I would have to look into that again, but I don't think it works with the Nixpkgs-provided GHC 9.4. I think this relates to the fact that Nixpkgs always forces the core libraries in the version that the compiler shipped with. IIRC we'd hit bounds for containers very quickly.

@nikita-volkov
Copy link
Author

Let's check. If any issues come up, they should be easy to fix.

Also please notice that this PR is just refactoring and doesn't deal with dependencies.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants