Skip to content
This repository has been archived by the owner on Jun 1, 2024. It is now read-only.

Added FormatStackTraceAsArray option to write StackTrace as an array #230

Merged
merged 8 commits into from
Apr 8, 2019

Conversation

oscarmorasu
Copy link
Contributor

@oscarmorasu oscarmorasu commented Mar 30, 2019

What issue does this PR address?
This PR fixes #229

Does this PR introduce a breaking change?
No. The new flag is false by default, which keeps the current behavior. If the flag is set to true the new behavior takes place.

Please check if the PR fulfills these requirements

  • The commit follows our guidelines
  • Unit Tests for the changes have been added (for bug fixes / features)

Other information:

@mivano
Copy link
Contributor

mivano commented Apr 1, 2019

Thanks for the PR. Would it be possible to add a unit test for this? And can you also list this option in the readme.md to explain to users why you want this option?

@oscarmorasu
Copy link
Contributor Author

@mivano, I added two unit tests.

Also, I changed the Configure the Sink wiki page, explaining the use of the flag. The README.md doesn't seem to have any content related to the Sink Options.

@mivano mivano merged commit f9f8077 into serilog-contrib:dev Apr 8, 2019
@mivano
Copy link
Contributor

mivano commented Apr 8, 2019

Nice, thanks. I merged it into dev, so beta packages should be available soon.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Format the Stack Trace as an array of strings
3 participants