danepitkin commented on PR #37097: URL: https://github.com/apache/arrow/pull/37097#issuecomment-1681304913
> > Numpydocs now correctly reports 72 cython functions are missing documentation for their parameters (Should be fixed in a separate PR IMO) > > Could you work on this before we merge this to avoid CI failures on main? Created https://github.com/apache/arrow/pull/37218 ! This fixes most of the issues. I figured we can merge that, rebase here, and then fix anything leftover, which will be manageable to do in this PR. We won't be able to verify the numpydocs test in that PR because it doesn't have cython 3. I tested it with cython 3 locally instead, but didn't see the exact same number of issues that are reported here. -- 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]
