Skip to content

fix: report db connection errors#348

Merged
psteinroe merged 2 commits intomainfrom
fix/db-conn-error
Apr 19, 2025
Merged

fix: report db connection errors#348
psteinroe merged 2 commits intomainfrom
fix/db-conn-error

Conversation

@psteinroe
Copy link
Copy Markdown
Collaborator

@psteinroe psteinroe commented Apr 16, 2025

this pr reports database errors back when running the type-checker. they are reported as WorkspaceError and abort the diagnostics pull. the idea is that the user directly sees what's wrong instead of just waiting for 10s to see only the syntax errors and then wondering why the performance is so bad.

fixes #344

Comment thread crates/pgt_typecheck/src/lib.rs Outdated
Copy link
Copy Markdown
Collaborator

@juleswritescode juleswritescode left a comment

Choose a reason for hiding this comment

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

very nice!!!

@psteinroe psteinroe merged commit 1adfd23 into main Apr 19, 2025
7 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.

Better reporting for database connection issues

2 participants