test: run CI on both linux and macos (#318)

* test: run CI on both linux and macos

Signed-off-by: MrCroxx <[email protected]>

* fix: fix build on macos

Signed-off-by: MrCroxx <[email protected]>

* fix: try fix ci run on targets

Signed-off-by: MrCroxx <[email protected]>

---------

Signed-off-by: MrCroxx <[email protected]>

Author: MrCroxx

.github/template/template.yml

@@ -36,7 +36,10 @@ jobs:
         uses: ludeeus/action-shellcheck@master
   rust-test:
     name: rust test with codecov
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
     steps:
       - name: Checkout
         uses: actions/checkout@v3

.github/workflows/main.yml

@@ -43,7 +43,10 @@ jobs:
         uses: ludeeus/action-shellcheck@master
   rust-test:
     name: rust test with codecov
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
     steps:
       - name: Checkout
         uses: actions/checkout@v3

.github/workflows/pull-request.yml

@@ -42,7 +42,10 @@ jobs:
         uses: ludeeus/action-shellcheck@master
   rust-test:
     name: rust test with codecov
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
     steps:
       - name: Checkout
         uses: actions/checkout@v3

foyer-experimental-bench/src/utils.rs

@@ -31,7 +31,8 @@ use std::path::{Path, PathBuf};
 use itertools::Itertools;
 use nix::{fcntl::readlink, sys::stat::stat};
 
-#[cfg_attr(not(target_os = "linux"), expect(dead_code))]
+// TODO(MrCroxx): use `expect` after `lint_reasons` is stable.
+#[cfg_attr(not(target_os = "linux"), allow(dead_code))]
 #[derive(PartialEq, Clone, Copy, Debug)]
 pub enum FsType {
     Xfs,
@@ -41,7 +42,8 @@ pub enum FsType {
     Others,
 }
 
-#[cfg_attr(not(target_os = "linux"), expect(unused_variables))]
+// TODO(MrCroxx): use `expect` after `lint_reasons` is stable.
+#[cfg_attr(not(target_os = "linux"), allow(unused_variables))]
 pub fn detect_fs_type(path: impl AsRef<Path>) -> FsType {
     #[cfg(target_os = "linux")]
     {

foyer-storage-bench/src/utils.rs

@@ -31,7 +31,8 @@ use std::path::{Path, PathBuf};
 use itertools::Itertools;
 use nix::{fcntl::readlink, sys::stat::stat};
 
-#[cfg_attr(not(target_os = "linux"), expect(dead_code))]
+// TODO(MrCroxx): use `expect` after `lint_reasons` is stable.
+#[cfg_attr(not(target_os = "linux"), allow(dead_code))]
 #[derive(PartialEq, Clone, Copy, Debug)]
 pub enum FsType {
     Xfs,
@@ -41,7 +42,8 @@ pub enum FsType {
     Others,
 }
 
-#[cfg_attr(not(target_os = "linux"), expect(unused_variables))]
+// TODO(MrCroxx): use `expect` after `lint_reasons` is stable.
+#[cfg_attr(not(target_os = "linux"), allow(unused_variables))]
 pub fn detect_fs_type(path: impl AsRef<Path>) -> FsType {
     #[cfg(target_os = "linux")]
     {

foyer-storage/src/device/fs.rs

@@ -62,7 +62,8 @@ impl FsDeviceConfig {
 struct FsDeviceInner {
     config: FsDeviceConfig,
 
-    #[cfg_attr(not(target_os = "linux"), expect(dead_code))]
+    // TODO(MrCroxx): use `expect` after `lint_reasons` is stable.
+    #[cfg_attr(not(target_os = "linux"), allow(dead_code))]
     dir: File,
 
     files: Vec<File>,