fix: serde(skip) in not serde features

Author: ahabhgk

.github/workflows/ci.yml

@@ -66,6 +66,7 @@ jobs:
           toolchain: 'stable'
           profile: minimal
           override: true
+      - run: cargo build
       # Since cargo test will run tests in parallel, and the crate uses cwd in
       # legacy importer and captured stdio for testing logger, so we need to
       # run tests in sequentially by adding `--test-threads=1`

src/api.rs

@@ -28,12 +28,12 @@ pub struct Options {
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/Options#alertColor)
   pub alert_color: Option<bool>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/Options#importers)
-  #[serde(skip)]
+  #[cfg_attr(feature = "serde", serde(skip))]
   pub importers: Vec<SassImporter>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/Options#loadPaths)
   pub load_paths: Vec<PathBuf>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/Options#logger)
-  #[serde(skip)]
+  #[cfg_attr(feature = "serde", serde(skip))]
   pub logger: Option<BoxedLogger>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/Options#quietDeps)
   pub quiet_deps: bool,
@@ -193,7 +193,7 @@ pub struct StringOptions {
   /// Field for [Options]
   pub common: Options,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/StringOptionsWithImporter#importer)
-  #[serde(skip)]
+  #[cfg_attr(feature = "serde", serde(skip))]
   pub input_importer: Option<SassImporter>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/StringOptionsWithoutImporter#syntax)
   ///  - [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/StringOptionsWithImporter#syntax)

src/legacy/api.rs

@@ -358,7 +358,7 @@ pub struct LegacyOptions {
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/LegacySharedOptions#sourceMapContents)
   pub source_map_contents: bool,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/LegacySharedOptions#importer)
-  #[serde(skip)]
+  #[cfg_attr(feature = "serde", serde(skip))]
   pub importers: Option<Vec<BoxedLegacyImporter>>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/LegacySharedOptions#charset)
   pub charset: bool,
@@ -367,7 +367,7 @@ pub struct LegacyOptions {
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/LegacySharedOptions#verbose)
   pub verbose: bool,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/LegacySharedOptions#logger)
-  #[serde(skip)]
+  #[cfg_attr(feature = "serde", serde(skip))]
   pub logger: Option<BoxedLogger>,
   /// More information: [Sass documentation](https://sass-lang.com/documentation/js-api/interfaces/LegacyFileOptions#file)
   pub file: Option<PathBuf>,