-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Errors during generation are non-existent #8366
Comments
I am seeing the same issue, errors are no longer being displayed. The only thing I get is the following statement |
@chriswetterman, I tested a similar setup locally.
|
Hi @charlypoly, I added the verbose flag and didn’t see any change in the output as it’s still truncated. Again this is in VS Code and I tried in both powershell and bash terminals. |
@chriswetterman, okay, we will investigate further. @wesselvdv, do you also face this issue on Windows or on another OS? |
two thumbs way up |
@charlypoly We have another project that's on [email protected] and it reports a plethora of errors as expected and I've rolled back to that for now. I just tested on [email protected] and the issue still persists (our Nexus repo doesn't have 2.13.5 yet to test). Currently don't have the bandwidth to narrow down exactly which release the issue began with but hopefully this gives you some more info. I'll circle back when things free up on my end. Thanks. |
It appears |
Hi @hannahborel, @wesselvdv, Could you confirm that you also face this bug only on Windows? |
@charlypoly I can confirm downgrading the version to 2.9.0 is working well on Mac. If I install version 2.10.0 or above it doesn't work. |
I can also confirm that the issue occurs on macOS as well. Reverting the CLI to 2.9.0 does indeed fix it and reintroduces error information in addition to just "Unable to validate GraphQL document!". |
Hi all, The culprit is most likely the following commit: e2cfc5c However, reverting this commit will bring back another bug that printed all error messages twice. The root cause of the error truncating issue seems to be linked to our migration to Listr2, which I experienced, which seems to truncate all logging containing multiple lines. There is no easy fix regarding this issue, and we prioritize it as one of the first things to tackle as part of the v3 roadmap. |
Much appreciated, @charlypoly. Thank you and looking forward to v3. |
Hi @chriswetterman, hi all, Could you give a try to the following canary release of
It should fix the issue of error truncating on Windows and also most of the issues of error details on all platforms. |
@pleunv @wesselvdv @chriswetterman @hannahborel, any feedback with the canary version suggested above? |
@charlypoly I just encountered this problem, and tried out the changes you made in #8525, and they do appear to resolve the issues. |
A fix has been released in |
Apologies, completely missed this. Just want to confirm that this does in fact also fix the issue for us. Thank you! |
Describe the bug
I've recently updated to v2 of the CLI and the associated plugins. Since then the errors that are reported are not helpful in that only a partial of the error is displayed. This makes tracking down errors extremely more challenging. Please see the below screenshots and information. I have used the
debug
anderrorsOnly
flags and they do not provide any additional information toward identifying the reason and therefore cause of the failure.Your Example Website or App
None provided
Steps to Reproduce the Bug or Issue
To reproduce I have the following CLI version and plugins installed which at the time are the most current versions. Create a query/mutation that is incorrect or doesn't exist to cause any error in generation. Review the reported errors.
Expected behavior
As a user, I am expecting to see detailed enough errors to allow me to track down my errors to resolve. Version 1 of the CLI provided adequate information around errors.
Screenshots or Videos
Platform
To reproduce I have the following CLI version and plugins installed which at the time are the most current versions.
Codegen Config File
Additional context
No response
The text was updated successfully, but these errors were encountered: