-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sort.cc added to runtime for nms compatability #7942
Conversation
@AlexanderSerov please resolve the lint error |
Update PR but it not solve the problem, checks locally. Can't understand what's wrong |
@AlexanderSerov please run clang-format-10 on |
@masahi I did so. This tool just delete a problem line and don't say why. Also I tried to make changes from different text redactors - the same. If you understand linting, can you resolve the issue instead of me as this will be a lot faster. This issues marked as contributors can make change. |
It is just complaining that file names are not sorted in alphabet order. |
@masahi Done. Thanks |
Thanks @AlexanderSerov |
* sort.cc added to runtime for nms compatability * remove include * fix clang lint * sort includes in alphabet order Co-authored-by: Alexander Serov <alexander@tech5.com>
* sort.cc added to runtime for nms compatability * remove include * fix clang lint * sort includes in alphabet order Co-authored-by: Alexander Serov <alexander@tech5.com>
sort.cc added to wasn runtime for nms support
Solves this issue