chore: construct roles vector via map (#766)

sweb · web-flow · commit 44bfedde57ca · 2026-03-30T11:21:29.000Z
diff --git a/rust/operator-binary/src/crd/mod.rs b/rust/operator-binary/src/crd/mod.rs
@@ -753,11 +753,7 @@ impl AirflowRole {
     }
 
     pub fn roles() -> Vec<String> {
-        let mut roles = vec![];
-        for role in Self::iter() {
-            roles.push(role.to_string())
-        }
-        roles
+        Self::iter().map(|r| r.to_string()).collect()
     }
 
     pub fn listener_class_name(&self, airflow: &v1alpha2::AirflowCluster) -> Option<String> {

Original file line number	Diff line number	Diff line change
`@@ -753,11 +753,7 @@ impl AirflowRole {`
`753`	`753`	`}`
`754`	`754`
`755`	`755`	`pub fn roles() -> Vec<String> {`
`756`		`- let mut roles = vec![];`
`757`		`- for role in Self::iter() {`
`758`		`- roles.push(role.to_string())`
`759`		`- }`
`760`		`- roles`
	`756`	`+ Self::iter().map(\|r\| r.to_string()).collect()`
`761`	`757`	`}`
`762`	`758`
`763`	`759`	`pub fn listener_class_name(&self, airflow: &v1alpha2::AirflowCluster) -> Option<String> {`