CI Warnings and use of actions-rs actions about libseccomp-rs HOT 5 CLOSED

@ManaSugi since you have more knowledge with tarpaulin, I would leave it to you to replace the tarpaulin action.

from libseccomp-rs.

@rusty-snake Thank you for the information!

As you mentioned, actions-rs/toolchain does not seem to be maintained for a long time, so we may have to start considering
an alternative action, just using rustup or stuff like that in order to avoid future problems.
On the other hand, the actions-rs/tarpaulin seems to be maintained actively, so I hope the mainteiner will handle this problem.
Could you create the issue for actions-rs/tarpaulin in order to tell it to the maintainers when you have time?

from libseccomp-rs.

cargo-tarpaulin itself is maintained. However we use it through the actions-rs/tarpaulin action.
It has three commits in the last two years, but if you wait two days (timezone ...) it will have zero commits in the last two years 😀 .

actions-rs has one public member, who has a lot activity in 2019/2020 and very less in 2021/2022.

For toolchain we could use https://github.com/marketplace/actions/rustup-toolchain-install.

For tarpaulin we need to write our own bash/python script, call it directly or find an other action. grcov does not seem to have an action either.

from libseccomp-rs.

cargo-tarpaulin itself is maintained. However we use it through the actions-rs/tarpaulin action.

Oh, you're right.

For toolchain we could use https://github.com/marketplace/actions/rustup-toolchain-install.

That's good an alterative action! We could use it instead of actions-rs/toolchain.
Regarding actions-rs/tarpaulin, I think we might as well write our own bash script in build-test.yaml directly. That won't be so many lines. In addition, It would be nice to switch to source-based coverage of cargo tarpaulin in the future in order to get more accurate coverage when it is available.
Ref. https://github.com/xd009642/tarpaulin/issues?q=is%3Aissue+is%3Aopen+%23549+

from libseccomp-rs.

This issue has been solved by the following commits.
5dcd06c
1748422

So, I can close this. Thanks! @rusty-snake

from libseccomp-rs.