chore: fix build script warning, fix doctest with default features

mbrobbel · Jul 9, 2024 · 3fe9f97 · 3fe9f97
1 parent 6fa602d
commit 3fe9f97
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/build.rs b/build.rs
@@ -280,7 +280,7 @@ fn main() -> Result<(), Box<dyn Error>> {
     // for use in docker build where file changes can be wonky
     println!("cargo:rerun-if-env-changed=FORCE_REBUILD");
 
-    let version = substrait_version()?;
+    let _version = substrait_version()?;
 
     #[cfg(feature = "protoc")]
     std::env::set_var("PROTOC", protobuf_src::protoc());
@@ -290,7 +290,7 @@ fn main() -> Result<(), Box<dyn Error>> {
     text(out_dir.as_path())?;
 
     #[cfg(feature = "extensions")]
-    extensions(version, out_dir.as_path())?;
+    extensions(_version, out_dir.as_path())?;
 
     let protos = WalkDir::new(PROTO_ROOT)
         .into_iter()

diff --git a/src/lib.rs b/src/lib.rs
@@ -69,6 +69,7 @@
 //!
 //! #### Read a simple extension
 //! ```rust
+//! # #[cfg(feature="extensions")]
 //! # fn main() -> Result<(), serde_yaml::Error> {
 //! use substrait::text::simple_extensions::SimpleExtensions;
 //!
@@ -96,6 +97,8 @@
 //! assert_eq!(simple_extension.scalar_functions.len(), 1);
 //! assert_eq!(simple_extension.scalar_functions[0].name, "add");
 //! # Ok(()) }
+//! # #[cfg(not(feature="extensions"))]
+//! # fn main() {}
 //! ```
 //!
 //! [pbjson]: https://docs.rs/pbjson