Skip to content

Conversation

fxamacker
Copy link
Member

@fxamacker fxamacker commented Jul 24, 2025

This PR adds more information that is useful for debugging.

Context: https://flow-foundation.slack.com/archives/C07NFGGAGHM/p1753382098661809?thread_ts=1753310325.017179&cid=C07NFGGAGHM

NOTE: If we want to merge this, then I need to fix/update tests before merging.


  • Targeted PR against main branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@fxamacker fxamacker requested a review from turbolent as a code owner July 24, 2025 18:32
Copy link
Member

@turbolent turbolent left a comment

Choose a reason for hiding this comment

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

It would be nice to merge this, it seems useful to have when debugging 👍

@fxamacker fxamacker changed the title [DO NOT MERGE] Return more data from String() for debugging Return more data from String() for debugging Sep 26, 2025
@fxamacker fxamacker self-assigned this Sep 26, 2025
@fxamacker fxamacker merged commit 2c25e45 into main Sep 26, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants