tmoreau89 commented on pull request #32: URL: https://github.com/apache/tvm-vta/pull/32#issuecomment-912250501
Thanks @vegaluisjose for reviewing the PR and ensuring that the CI tests go green, much appreciated. While going through the PR there were a few things that I'd like to address before we merge: - Still quite a few comments that need to be cleaned up - The comments could use a good spell check pass, many typos in some files - perhaps it could be automated via the preferred IDE - Overall I think this is not blocking the merge, but we could extend the scala linter to address consistent comment formatting etc. -- 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]
