Skip to content

Commit a4af6ea

Browse files
adwk67maltesander
andauthored
Apply suggestions from code review
Co-authored-by: maltesander <malte.sander.it@gmail.com>
1 parent f648a2d commit a4af6ea

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

crates/stackable-operator/src/v2/flask_config_writer.rs

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,5 @@
11
//! Writer for Flask App configurations (Python config files).
22
//!
3-
//! Originally `product_config::flask_app_config_writer`; previously vendored into
4-
//! airflow- and superset-operator, now provided here as the shared home so operators
5-
//! do not depend on the `product-config` crate. Applications based on the Flask App
6-
//! Builder (e.g. Apache Airflow, Apache Superset) use configuration files written in
7-
//! Python. This writer only covers top-level assignments of a few primitive types and
8-
//! expressions — it is not a general Python code generator.
9-
//!
103
//! Primitive types are escaped accordingly. Python expressions are written as-is;
114
//! invalid expressions produce invalid configuration files. Config overrides that do
125
//! not map to a known option are treated as plain expressions.
@@ -20,8 +13,6 @@ use std::{
2013
use snafu::{ResultExt, Snafu};
2114

2215
/// Errors which can occur when using this module
23-
// Variant names share the `Error` suffix; kept as-is from the vendored
24-
// `product_config::flask_app_config_writer` source.
2516
#[allow(clippy::enum_variant_names)]
2617
#[derive(Debug, Snafu)]
2718
pub enum FlaskAppConfigWriterError {

0 commit comments

Comments
 (0)