2b) add cpp and rust tests on ci #1073
Conversation
.github/workflows/ubuntu.yml
Outdated
| toolchain: stable | ||
|
|
||
| - 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 |
crop2000
changed the title
|
@sletz to run implulse tests in cpp and rust takes now 13min |
setup github action to run tests the file compiler/generator/libfaust.h needs to be copied to architecture/faust/dsp/ to be available in ci remove prefix.dsp ir from gcc tests remove constant.dsp ir from gcc mapp tests
3 participants
based on #1068
a lot of work was caused by #1075