freediag icon indicating copy to clipboard operation
freediag copied to clipboard

CI / continuous testing would be nice

Open fenugrec opened this issue 4 years ago • 4 comments

from PR #64 , the ability to test compilation on MSVC + borland + mingw automatically would be nice.

  • [ ] run diag_test binary for some internal tests
  • [ ] run the subset of ctest tests that don't require a hardware interface (currently ctest -I 3 ...)
  • [ ] running with valgrind for bonus points
  • [ ] cppcheck / other static checker for more bonus

fenugrec avatar Feb 11 '21 15:02 fenugrec

test compilation on MSVC + borland + mingw

  • Borland could be a bit of a problem probably because it may not be installed on the test runner.
  • Linux build should be tested as well I think.

running with valgrind for bonus points cppcheck / other static checker for more bonus

Yes that would be done in a secound step.

cfehse avatar Feb 12 '21 11:02 cfehse

@fenugrec

Which kind of approach would you prefer for the github action workflow:

A more "native" and "splitted" approach as for example here: https://github.com/danmar/cppcheck/tree/main/.github/workflows

Or a cmake based "all in one" definition as for example here: https://github.com/cristianadam/HelloWorld/tree/master/.github/workflows

cfehse avatar Feb 25 '21 14:02 cfehse

A more "native" and "splitted" approach as for example here:

If it's a comparable amount of work and functionality, I think I prefer the "split" approach - a bit easier to digest and work on, perhaps ? If there are reasons to prefer the second, I could be convinced.

fenugrec avatar Feb 27 '21 18:02 fenugrec

I just asked for personal preference. I would tend to "split" approach as well.

So decision made: 1 workflow definition per architecture.

cfehse avatar Feb 27 '21 18:02 cfehse