fix: clippy — collapse nested if, remove useless .into()
This commit is contained in:
@@ -21,7 +21,7 @@ pub fn run_extraction(
|
||||
) -> Result<AnalysisResult, DomainError> {
|
||||
let mut parser = Parser::new();
|
||||
parser
|
||||
.set_language(&extractor.tree_sitter_language().into())
|
||||
.set_language(&extractor.tree_sitter_language())
|
||||
.map_err(|e| DomainError::AnalysisError(e.to_string()))?;
|
||||
|
||||
let tree = parser
|
||||
|
||||
@@ -51,13 +51,13 @@ where
|
||||
let analyze = AnalyzeCodebase::new(self.discovery, self.source_analyzer);
|
||||
let result = analyze.execute(root, config)?;
|
||||
let mut graph = result.graph().clone();
|
||||
if level == DiagramLevel::Module {
|
||||
if let Some(pa) = self.project_analyzer {
|
||||
// Propagate error: a present but malformed manifest is a user error,
|
||||
// not a silent skip.
|
||||
let project_cg = pa.analyze(root)?;
|
||||
graph.merge_project_edges(&project_cg);
|
||||
}
|
||||
if level == DiagramLevel::Module
|
||||
&& let Some(pa) = self.project_analyzer
|
||||
{
|
||||
// Propagate error: a present but malformed manifest is a user error,
|
||||
// not a silent skip.
|
||||
let project_cg = pa.analyze(root)?;
|
||||
graph.merge_project_edges(&project_cg);
|
||||
}
|
||||
Ok(BuildCodeGraphResult {
|
||||
graph,
|
||||
|
||||
Reference in New Issue
Block a user