-
Notifications
You must be signed in to change notification settings - Fork 175
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
Imrpove CI #269
Comments
That would be good. Just tried to get it to compile with VS 2019/v142 with the existing appveyor, but seems to need more steps to enable toolsets/compiler versions. Edit - looks like I do have it working for VS 2015(v140) with C++14, and VS 2019 (v142) with C++20 and __cplusplus correct. |
#313 improved things but there's still some things to fix to properly be able to test MSVC. I'll probably remove the line "add more compilers" because I don't really want to invest a lot of time into supporting them. |
#317 fixed C++14 tests on MSVC. The only known issue remaining is with RTTI stuff on MSVC x64. |
With #318 now there's an action to run tests with sanitizers enabled. |
With #319 CI test with |
With #335 now tests are run for MSVC 64 bits (expect tests that depend on RTTI as it isn't properly supported in fakeit for this configuration yet). |
Take inspiration from range-v3 CI configuration, which looks great : https://github.com/ericniebler/range-v3/blob/master/.github/workflows/range-v3-ci.yml
The text was updated successfully, but these errors were encountered: