Skip to content

Commit beec382

Browse files
committed
fix: resolve clippy -D warnings across workspace
- azdo.rs: add doc comments to all public struct fields, enum variants, and associated functions (97 missing-doc errors) - runner-server: remove duplicate imports from merge conflict, fix unused variables, fix field-assignment-outside-initializer lint - parser: prefix unused functions, fix unused variable, use vec![] over push-after-creation
1 parent ef863be commit beec382

4 files changed

Lines changed: 108 additions & 24 deletions

File tree

‎crates/aksh-gha-parser/src/job_builder.rs‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,7 @@ pub fn build_agent_job_message(
9191
.collect();
9292

9393
// Service endpoints (SystemVssConnection)
94-
let mut endpoints = Vec::new();
95-
endpoints.push(ServiceEndpoint {
94+
let endpoints = vec![ServiceEndpoint {
9695
data: BTreeMap::new(),
9796
name: "SystemVssConnection".to_owned(),
9897
endpoint_type: Some("azdoserver".to_owned()),
@@ -109,7 +108,7 @@ pub fn build_agent_job_message(
109108
)]),
110109
is_shared: Some(false),
111110
service_owner: Some("github".to_owned()),
112-
});
111+
}];
113112

114113
let resources = TaskResources {
115114
endpoints,
@@ -178,7 +177,7 @@ pub fn build_agent_job_message(
178177
}
179178

180179
/// Build a `TaskStep` from a `StepPlan`.
181-
fn build_task_step(step: &crate::StepPlan, index: usize, context: &Context) -> TaskStep {
180+
fn build_task_step(step: &crate::StepPlan, _index: usize, context: &Context) -> TaskStep {
182181
let step_id = uuid::Uuid::new_v4();
183182

184183
// Resolve expressions in env and with

‎crates/aksh-gha-parser/src/lib.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -756,7 +756,7 @@ fn can_merge_include_indexed(
756756
.all(|(key, value)| candidate.get(key).is_none_or(|existing| existing == value))
757757
}
758758

759-
fn object_entry(
759+
fn _object_entry(
760760
job_id: &str,
761761
field: &'static str,
762762
value: &Value,
@@ -781,7 +781,7 @@ fn matches_partial(candidate: &IndexMap<String, Value>, partial: &IndexMap<Strin
781781
})
782782
}
783783

784-
fn can_merge_include(
784+
fn _can_merge_include(
785785
candidate: &IndexMap<String, Value>,
786786
include: &IndexMap<String, Value>,
787787
) -> bool {

0 commit comments

Comments
 (0)