OlivierJaquemet commented on PR #16446: URL: https://github.com/apache/lucene/pull/16446#issuecomment-5509295592
Yes, I agree, this looks purely cosmetic, and your explanation makes sense. I think we can safely rely on the existing test coverage and update the affected tests. Would the best next step be to close my two PRs and open a single one with both test changes and your proposed fix? If so, would you prefer to handle it and use your explanation in the PR, or should I take care of it? -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
