-
Notifications
You must be signed in to change notification settings - Fork 601
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
Compare Windows unit test results with Linux #34
Comments
|
I have updated the uncrustify wrapper package to work on windows (supposingly): ament/uncrustify@96f42bc @wjwwood Currently Visual Studio on the Windows slave does not work anymore. Once this is fixed please trigger another build to check if it makes the uncrustify tests pass. |
All uncrustify tests: passing |
Linux and Windows now have the same number of tests (530):
There are some skipped tests and failing tests on Windows, but we have the same number at least. I scan over the tests and it appears that all of the failing tests on Windows are present on Linux and that there is not any linux only or windows only tests. Closing this, please comment if you see a problem. |
Depends on #21
AC:
The text was updated successfully, but these errors were encountered: