Adds building and testing of examples to CI. #599
Conversation
Superhepper
force-pushed
the
Ci-Examples
branch
from
65e4dbe to
f0335b4
Compare
Superhepper
requested review from
Firstyear,
hug-dev,
ionut-arm and
wiktor-k
tss-esapi/tests/examples.sh
Outdated
| ################## | ||
| # Build examples # | ||
| ################## | ||
| RUST_BACKTRACE=1 cargo build --examples |
There was a problem hiding this comment.
I think this shouldn't be necessary as cargo test --examples needs to build them first anyway... 🤔
There was a problem hiding this comment.
No, but its nice to separate build errors from running the examples errors.
There was a problem hiding this comment.
No, but its nice to separate build errors from running the examples errors.
There was a problem hiding this comment.
Then I suggest using cargo check which shows build errors but doesn't waste time creating output artifacts (that we wouldn't need in CI anyway). 👋
There was a problem hiding this comment.
Yeah, that could be done.
Oh I just noted that the script does not start the tpm server ... But everything is green anyway ..
There was a problem hiding this comment.
Yeah, that could be done.
Oh I just noted that the script does not start the tom server ... But everything is green anyway ..
There was a problem hiding this comment.
This was way more complicated then I expected. The examples requires a resource broker.
Superhepper
force-pushed
the
Ci-Examples
branch
5 times, most recently
from
5cfecc7 to
e1fd179
Compare
Superhepper
force-pushed
the
Ci-Examples
branch
from
e1fd179 to
85e36c0
Compare
Signed-off-by: Jesper Brynolf <[email protected]>
Superhepper
force-pushed
the
Ci-Examples
branch
from
85e36c0 to
2807879
Compare
2 participants
Fixes #467