refactor: apply clippy rules

This commit is contained in:
Matthieu Bessat 2025-10-11 23:11:46 +02:00
parent 46df9f7aeb
commit 9e0f038847
3 changed files with 15 additions and 15 deletions

View file

@ -27,7 +27,7 @@ impl Field {
} }
/// Generate CREATE TABLE statement from parsed model /// Generate CREATE TABLE statement from parsed model
pub fn generate_create_table_sql(models: &Vec<Model>) -> Result<String> { pub fn generate_create_table_sql(models: &[Model]) -> Result<String> {
let mut sql_code: String = "".into(); let mut sql_code: String = "".into();
for model in models.iter() { for model in models.iter() {
let mut fields_sql: Vec<String> = vec![]; let mut fields_sql: Vec<String> = vec![];

View file

@ -217,18 +217,18 @@ fn gen_delete_by_id_method(model: &Model) -> TokenStream {
fn generate_repository_file(model: &Model) -> Result<SourceNodeContainer> { fn generate_repository_file(model: &Model) -> Result<SourceNodeContainer> {
let resource_name = model.name.clone(); let resource_name = model.name.clone();
let resource_module_ident = format_ident!("{}", &model.module_path.get(0).unwrap()); let resource_module_ident = format_ident!("{}", &model.module_path.first().unwrap());
let resource_ident = format_ident!("{}", &resource_name); let resource_ident = format_ident!("{}", &resource_name);
let repository_ident = format_ident!("{}Repository", resource_ident); let repository_ident = format_ident!("{}Repository", resource_ident);
let get_all_method_code = gen_get_all_method(&model); let get_all_method_code = gen_get_all_method(model);
let get_by_id_method_code = gen_get_by_id_method(&model); let get_by_id_method_code = gen_get_by_id_method(model);
let get_many_by_id_method_code = gen_get_many_by_id_method(&model); let get_many_by_id_method_code = gen_get_many_by_id_method(model);
let insert_method_code = gen_insert_method(&model); let insert_method_code = gen_insert_method(model);
let insert_many_method_code = gen_insert_many_method(&model); let insert_many_method_code = gen_insert_many_method(model);
let update_by_id_method_code = gen_update_by_id_method(&model); let update_by_id_method_code = gen_update_by_id_method(model);
let delete_by_id_method_code = gen_delete_by_id_method(&model); let delete_by_id_method_code = gen_delete_by_id_method(model);
// TODO: add import line // TODO: add import line
@ -288,7 +288,7 @@ struct SourceNodeContainer {
} }
/// Generate base repositories for all models /// Generate base repositories for all models
pub fn generate_repositories_source_files(models: &Vec<Model>) -> Result<SourceNodeContainer> { pub fn generate_repositories_source_files(models: &[Model]) -> Result<SourceNodeContainer> {
let mut nodes: Vec<SourceNodeContainer> = vec![]; let mut nodes: Vec<SourceNodeContainer> = vec![];
for model in models.iter() { for model in models.iter() {
let snc = generate_repository_file(model)?; let snc = generate_repository_file(model)?;

View file

@ -69,7 +69,7 @@ fn parse_model_attribute(item: &syn::ItemStruct) -> Result<Option<SqlGeneratorMo
continue; continue;
} }
}; };
if attr_ident.to_string() != "sql_generator_model" { if *attr_ident != "sql_generator_model" {
continue; continue;
} }
@ -93,7 +93,7 @@ fn parse_field_attribute(field: &syn::Field) -> Result<Option<SqlGeneratorFieldA
continue; continue;
} }
}; };
if attr_ident.to_string() != "sql_generator_field" { if *attr_ident != "sql_generator_field" {
continue; continue;
} }
@ -111,10 +111,10 @@ fn parse_field_attribute(field: &syn::Field) -> Result<Option<SqlGeneratorFieldA
/// Take struct name as source, apply snake case and pluralize with a s /// Take struct name as source, apply snake case and pluralize with a s
fn generate_table_name_from_struct_name(struct_name: &str) -> String { fn generate_table_name_from_struct_name(struct_name: &str) -> String {
return format!( format!(
"{}s", "{}s",
struct_name.to_case(Case::Snake) struct_name.to_case(Case::Snake)
); )
} }
/// Scan for models struct in a rust file and return a struct representing the model /// Scan for models struct in a rust file and return a struct representing the model
@ -221,7 +221,7 @@ pub fn parse_models_from_module(module_path: &Path) -> Result<Vec<Model>> {
let mut models: Vec<Model> = vec![]; let mut models: Vec<Model> = vec![];
if module_path.is_file() { if module_path.is_file() {
models.extend(parse_models(&module_path)?); models.extend(parse_models(module_path)?);
return Ok(models); return Ok(models);
} }