Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error visibility in query file generation #866

Open
jackfischer opened this issue Feb 15, 2024 · 2 comments
Open

Error visibility in query file generation #866

jackfischer opened this issue Feb 15, 2024 · 2 comments
Assignees

Comments

@jackfischer
Copy link

Describe the proposed feature
A clear and concise description of the syntax, what you want to happen, and why it's needed.

Query files generated from @edgedb/generate queries seem to generate in alphabetical order. With several hundred queries, any errors from the particular file you're iterating on are buried somewhere in pages of output. It would be helpful if errors could somehow be surfaced at the bottom of the output. Maybe in conjunction with #865, write all the output for successful queries first and then all errors at the end.

@scotttrinh scotttrinh self-assigned this Feb 15, 2024
@scotttrinh
Copy link
Collaborator

I have a crazy idea: what if the output was TAP? That way users would be able to use one of the many TAP reporters to format the output here? Then you could also have a different output format for CI, etc.

Feels like a bit of a stretch to use a testing-specific format for this, but when I was thinking about this problem from a data structure perspective (({ path } | { path, error })[]) it started to look suspiciously like TAP.

@jackfischer
Copy link
Author

Wasn't familiar with TAP but indeed that data structure sounds exactly right!

themajashurka added a commit to themajashurka/edgedb-js that referenced this issue Feb 21, 2024
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

No branches or pull requests

2 participants