d-e-s-o / nitrocli

A command line tool for interacting with Nitrokey devices.
30 stars 10 forks source link

Add GitHub Actions configuration #153

Closed robinkrahl closed 3 years ago

robinkrahl commented 3 years ago

This patch adds a configuration for GitHub Actions that builds and tests nitrocli with the MSRV, stable, beta and nightly Rust, calls clippy and rustfmt and checks the license annotations with reuse-tool.

(Previous discussion at #148.)

robinkrahl commented 3 years ago

Build logs for this PR.

robinkrahl commented 3 years ago

Notes:

d-e-s-o commented 3 years ago

Fabulous stuff! Thanks for looking into that.

We can do that. Let's first move forward with what we have. It seems to already be a hefty speed boost.

  • Most examples use cargo check instead of cargo build. But as our build involves linking to a shared library (which AFAIK is skipped for check), I think performing the full build is better for us.

:+1:

d-e-s-o commented 3 years ago

Will merge tomorrow.

d-e-s-o commented 3 years ago

Merged.

robinkrahl commented 3 years ago

Thanks for merging! By the way, I also tested macOS, but we get an exception when trying to run the tests.

d-e-s-o commented 3 years ago

Thanks for merging! By the way, I also tested macOS, but we get an exception when trying to run the tests.

Hm, interesting. No idea how to debug this issue, to be honest. We may still need someone with a macOS at hand.