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

fix: use debugValueToString for debugger operations #759

Merged
merged 1 commit into from
Jun 11, 2024

Conversation

theSuess
Copy link
Contributor

The valueToString operation introduced by #742 is incompatible with the way the implementation from #739 as it tries to manifest an object from stack while the implementation needed by the debugger returns the value as-is without further evaluation.

The `valueToString` operation introduced by google#742 is incompatible with the way
the implementation from google#739 as it tries to manifest an object from stack while
the implementation needed by the debugger returns the value as-is without
further evaluation.
@sbarzowski sbarzowski merged commit 2b4d753 into google:master Jun 11, 2024
8 checks passed
@sbarzowski
Copy link
Collaborator

Thank you!

@coveralls
Copy link

Coverage Status

coverage: 68.143% (-0.4%) from 68.497%
when pulling 8a3c380 on theSuess:fix/add-debugvaluetostring
into 214a617 on google:master.

vhata pushed a commit to discord/go-jsonnet that referenced this pull request Aug 30, 2024
The `valueToString` operation introduced by google#742 is incompatible with the way
the implementation from google#739 as it tries to manifest an object from stack while
the implementation needed by the debugger returns the value as-is without
further evaluation.
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.

3 participants