Conversation
| Assert.True(array.GetValue(7).HasValue); | ||
| Assert.True(array.GetValue(7).Value); | ||
| #pragma warning disable CS0618 | ||
| #pragma warning disable CS0618 |
There was a problem hiding this comment.
I think this is fine. I'd probably prefer the original format with indent but it doesn't look like that's supported by dotnet format, although JetBrains Resharper has a custom extension to support this: https://www.jetbrains.com/help/resharper/EditorConfig_CSHARP_CSharpIndentStylePageSchema.html#resharper_csharp_indent_preprocessor_other
| Assert.True(array.GetValue(7).HasValue); | ||
| Assert.True(array.GetValue(7).Value); | ||
| #pragma warning disable CS0618 | ||
| #pragma warning disable CS0618 |
There was a problem hiding this comment.
I think this is fine. I'd probably prefer the original format with indent but it doesn't look like that's supported by dotnet format, although JetBrains Resharper has a custom extension to support this: https://www.jetbrains.com/help/resharper/EditorConfig_CSHARP_CSharpIndentStylePageSchema.html#resharper_csharp_indent_preprocessor_other
Co-authored-by: Adam Reeve <adreeve@gmail.com>
2 participants
What's Changed
Run
dotnet formatvia pre-commit.This excludes
src/Apache.Arrow/Flatbuf/FlatBuffers/because it's a bundled source codes.See also: #18
Closes #2.