search

seanmonstar / num_cpus

Get the number of CPUs in Rust
Apache License 2.0
582 stars 91 forks source link

ci: set up github action workflow for rust #114

Closed PureWhiteWu closed 2 years ago

PureWhiteWu commented 3 years ago

image

You may need to do a one-time setup for actions and remove the travis.

seanmonstar commented 3 years ago

Thanks for starting this! It'd be helpful if we could carry over all the different OS/targets that are checked in Travis, since this crate is full of OS-specific code.

PureWhiteWu commented 3 years ago

Thanks for starting this! It'd be helpful if we could carry over all the different OS/targets that are checked in Travis, since this crate is full of OS-specific code.

Hello, Windows and macOS support is added.

PureWhiteWu commented 3 years ago

Hi, is there anything else need to do?

seanmonstar commented 3 years ago

Ideally, we could continue to test on the different targets that are seen in the .travis.yml file, and keep the container/cgroups tests as well.

PureWhiteWu commented 3 years ago

Hello, could you please merge this and #113 , and publish a new version? We are having problems in our production environment. @seanmonstar

PureWhiteWu commented 3 years ago

Hello, I've added all the target build tests in .travis.yml and some additional targets which Rust has Tier1 or Tier2 support. @seanmonstar The tests are now all successful, you may take a look at https://github.com/PureWhiteWu/num_cpus/pull/1

PureWhiteWu commented 3 years ago

ping~ @seanmonstar