Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jdegoes
Copy link
Member

@jdegoes jdegoes commented Nov 13, 2019

No description provided.

iravid
iravid previously approved these changes Nov 13, 2019
Copy link
Member

@iravid iravid left a comment

Choose a reason for hiding this comment

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

❤️😍😍

Copy link
Member

@iravid iravid left a comment

Choose a reason for hiding this comment

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

at some point it’d also be nice to get something that is machine parseable. For example, new line delimited json; one line per fiber. That works very well with tools like jq.

@jdegoes
Copy link
Member Author

jdegoes commented Nov 13, 2019

@iravid Agreed! We should integrate that into zmx, a machine readable mode for all the dumps and metrics.

@softinio
Copy link
Contributor

@iravid Agreed! We should integrate that into zmx, a machine readable mode for all the dumps and metrics.

I'll work on that next week

softinio
softinio previously approved these changes Nov 13, 2019
}

s"""
${name}#${fiberId.seqNumber} (${lifeMsg}) ${waitMsg}}
Copy link
Member

Choose a reason for hiding this comment

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

Don't you have to use stripMargin here?

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you! 🙏

Copy link
Member

@regiskuckaertz regiskuckaertz Nov 13, 2019

Choose a reason for hiding this comment

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

No worries 🤗 You will have to prefix with | every line though, like this:

s"""
   |${name}#${fiberId.seqNumber} (${lifeMsg}) ${waitMsg}}
   |  Status: ${statMsg}
   |${trace.prettyPrint}
   |""".stripMargin

@jdegoes jdegoes merged commit cbde285 into zio:master Nov 13, 2019
@jdegoes jdegoes deleted the zstack branch November 13, 2019 12:59
Twizty pushed a commit to Twizty/zio that referenced this pull request Nov 18, 2019
* Add pretty print for dump

* Small tweaks to pretty printing

* Strip Margin
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.

4 participants