2b) add cpp and rust tests on ci #1073
Conversation
crop2000
force-pushed
the
ci_tests
branch
4 times, most recently
from
e5ca800
to
a148222
Compare
.github/workflows/ubuntu.yml
Outdated
|
|
||
| - name: Run impulse tests (C++) | ||
| run: cd tests/impulse-tests && make travis |
There was a problem hiding this comment.
Strange that the target is called "travis" (tests don't run on Travis CI, no?). But of course that's somewhat independent of this PR, if it happens to be called like this already.
There was a problem hiding this comment.
It is a subset of the cpp tests that finishes in below 1h
|
@sletz what do you think about this? |
|
This is too long, I suggest modifying the |
|
@sletz is the travis target used elsewhere? if not i would rename it to github_action and describe a meaning full subset. maybe it makes sense to have a detailed check that is not part of the default actions, but can be activated to ensure a thorough check for more complex changes. |
|
@sletz down to ~20m for everything ~5min build faust, ~15min c++ and rust |
setup github action to run tests the tests needed some changed to work on github remove prefix ir from gcc tests remove constant.dsp ir from gcc mapp tests improve rust impule_test makefile
crop2000
changed the title
based on #1068
a lot of work was caused by #1075