zhreshold commented on issue #11475: Amending ONNX importer/exporter #11213 URL: https://github.com/apache/incubator-mxnet/issues/11475#issuecomment-401410303 @marcoabreu If you think a parent folder for all pytest is necessary and reserved for future, I can take that. But isn’t pytest itself indicating python tests? IMO pytest is better than python-pytest as a parent folder name in this case. I always insist that a good naming and reasonable structure is critical to a project’ capability of maintenance and allow external contributors to better understand what has been done without asking “what is it?”. Therefore every decision had to be made with cautious otherwise it’s so hard to change later. I understand your situation that some testing facility may already depend on this, if you insist, I can hold the change as it’s a minor issue. But I am afraid there will never be a good chance to fix it in the future
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
