From 5addb84bfbeba78a350862ded5b4aa3d676cac61 Mon Sep 17 00:00:00 2001 From: drbh Date: Tue, 20 Feb 2024 10:38:35 -0500 Subject: [PATCH] fix: refactor syntax to correctly include structs (#1580) This PR fixes a compilation bug related to conditionally adding docs behind a feature flag --- router/src/server.rs | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/router/src/server.rs b/router/src/server.rs index 70a8015f..f634b49c 100644 --- a/router/src/server.rs +++ b/router/src/server.rs @@ -1041,20 +1041,21 @@ pub async fn run( }; // Define VertextApiDoc conditionally only if the "google" feature is enabled - #[cfg(feature = "google")] - #[derive(OpenApi)] - #[openapi( - paths(vertex_compatibility), - components(schemas(VertexInstance, VertexRequest, VertexResponse)) - )] - struct VertextApiDoc; - let doc = { // avoid `mut` if possible #[cfg(feature = "google")] { + use crate::VertexInstance; + + #[derive(OpenApi)] + #[openapi( + paths(vertex_compatibility), + components(schemas(VertexInstance, VertexRequest, VertexResponse)) + )] + struct VertextApiDoc; + // limiting mutability to the smallest scope necessary - let mut doc = doc; + let mut doc = ApiDoc::openapi(); doc.merge(VertextApiDoc::openapi()); doc }