uschindler commented on pull request #582: URL: https://github.com/apache/lucene/pull/582#issuecomment-1005597114
> > The name does not matter, only the suffix `.tests` matters. > > It kind of matters, I just think we should do this consistently. If you really like `integration.tests` name, that is fine, but please also fix the ones in `core` and `distribution` to match this scheme? There's a bit of difference: There is no "analysis" module. The other modules like core have core.tests (i dont like the name, but Dawid has chosen it). Sure, I can move the classes around no issue. But then analysis.test would be top-level next to the analysis subdirectory. If this is fine to you, I will do. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
