av8or1 commented on PR #39840: URL: https://github.com/apache/arrow/pull/39840#issuecomment-1923971210
> Could you run formatter? > > https://github.com/apache/arrow/actions/runs/7744737025/job/21135532299?pr=39840#step:5:3012 > > ``` > --- /arrow/cpp/src/arrow/filesystem/azurefs_test.cc > +++ /arrow/cpp/src/arrow/filesystem/azurefs_test.cc (after clang format) > @@ -1887,5 +1887,5 @@ > ASSERT_RAISES(Invalid, stream->ReadAt(1, 1)); > ASSERT_RAISES(Invalid, stream->Seek(2)); > } > -} // namespace fs > -} // namespace arrow > +} // namespace fs > +} // namespace arrow > /arrow/cpp/src/arrow/filesystem/azurefs_test.cc had clang-format style issues > ``` Yeah, funny thing that. During my first submission attempts, I encountered errors that complained about the fact that there were two spaces after the closing brace and prior to the namespace comments. Therefore I decided to delete one of those spaces on each line. After doing that, we see this error. It doesn't seem to like either implementation. I will however see what this formatter generates and then cross fingers. Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
