feat: System checks on `SystemRequirements` (cargo, rustc, msrv)

albersonmiranda commented 2 months ago

What type of PR is this? (check all applicable)

[x] New Feature
[ ] Bug Fix
[ ] Documentation Update
[ ] Style Update
[ ] Refactor
[ ] Tests
[ ] Other tasks

Description

This PR updates use_cran_defaults() to check SystemRequirements field in DESCRIPTION and perform a system check for both Rust and Cargo tools. If any of them is not found, build fails with a message to install them. If they are found, it checks for the minimum version of rustc. If it is lower than specified in SystemRequirements, build fails with a message stating both installed and minimum version required. Finally, if all tests pass, it prints the version of cargo and rustc found, which will be used to build the package.

Related Tickets & Documents

Closes #378

Added/updated tests?

Goal is code coverage percentage at 80% and above.

[x] Yes
[ ] No, and this is why: please replace this line with details on why tests have not been included
[ ] I need help with writing tests

Release checklist

[x] Bump version
[x] Update NEWS.md

CGMossa commented 2 months ago

Just wanted to say, this looks impressive already. Keep up the good work @albersonmiranda!

albersonmiranda commented 2 months ago

@Ilia-Kosenkov After https://github.com/extendr/extendr/pull/844 it is now passing R CMD Check locally and it's ready to review.

Ilia-Kosenkov commented 2 months ago

I got it that the version in DESCRIPTION is annoying for snapshot tests, I will fix it after you merge this.

extendr / rextendr