Skip to content

Conversation

WAcry
Copy link
Contributor

@WAcry WAcry commented Oct 30, 2024

fixes #2019

developed based on #2035

To support multi dimensional arrays

Copy link
Member

@nblumhardt nblumhardt left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks @WAcry (and @sungam3r 👍)

CI is failing with some xUnit analyzer issues which the diff view also shows inline.

Otherwise just some minor naming nitpicks - LGTM otherwise.

@@ -317,6 +317,175 @@ public void NullMessageTemplateParametersDoNotBreakBinding()
// ReSharper restore StructuredMessageTemplateProblem
}

// https://github.com/serilog/serilog/issues/2019
[Fact]
public void Two_Dimensional_Array_Should_Be_Logger_As_Sequence()
Copy link
Member

Choose a reason for hiding this comment

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

Nit: _Should_Be_Logger => _Should_Be_Logged typo in this and several other of the test names.

Naming: tests in this project use PascalCase names, so TwoDimensionalArrayShouldBeLoggedAsSequence would be the matching casing here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed Naming and Linting

Copy link
Member

Choose a reason for hiding this comment

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

Thanks! Merged 👍

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.

Logging multidimensional 2d array fails silently
3 participants