Skip to content

Print meaningfull error message#180

Merged
rabbiveesh merged 1 commit intodbsrgits:masterfrom
KES777:provide_meaningfull_error
Mar 23, 2025
Merged

Print meaningfull error message#180
rabbiveesh merged 1 commit intodbsrgits:masterfrom
KES777:provide_meaningfull_error

Conversation

@KES777
Copy link
Copy Markdown
Contributor

@KES777 KES777 commented Mar 23, 2025

Problem

During tests we get unclear error message:

/home/kes/work/projects/github-forks/sql-translator/t/data/roundtrip_autogen.yaml: No such file or directory at t/60roundtrip.t line 96.

It is problematic to understand what is the file and how to get it.

Solution

Provide the instruction to user how to regenerate this file. The similar attempt we can see from PostgreSQL error messages where HINT provides some direction to users to resolve a problem.

Copy link
Copy Markdown
Contributor

@rabbiveesh rabbiveesh left a comment

Choose a reason for hiding this comment

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

thanks; these small QOL things are huge in the big scheme of things.

@rabbiveesh rabbiveesh merged commit 18be6d1 into dbsrgits:master Mar 23, 2025
5 checks passed
@KES777 KES777 deleted the provide_meaningfull_error branch September 8, 2025 00:43
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