diff --git a/src/references.rs b/src/references.rs index 5eb6f9e..c14da9e 100644 --- a/src/references.rs +++ b/src/references.rs @@ -56,8 +56,8 @@ impl ReferenceSignatures { config: &tracker::Config, ) -> Result { let mut signatures: HashMap, Vec> = HashMap::new(); - Self::store(&mut signatures, ref_bugs, &config)?; - Self::store(&mut signatures, ref_issues, &config)?; + Self::store(&mut signatures, ref_bugs, config)?; + Self::store(&mut signatures, ref_issues, config)?; // For each ticket, sort its references alphabetically. // Otherwise, the order changes based on the response from the ticket tracker, diff --git a/src/status_report.rs b/src/status_report.rs index 84b1f27..8836fc5 100644 --- a/src/status_report.rs +++ b/src/status_report.rs @@ -489,7 +489,7 @@ fn combined_releases(tickets: &[AbstractTicket]) -> Vec<&str> { // Releases are a list, and each ticket can have several of them. // Update the counter with the values in the lists, rather than // with the lists themselves as values. - for ticket in tickets.iter() { + for ticket in tickets { releases.update(ticket.target_releases.iter().map(String::as_str)); } diff --git a/src/templating.rs b/src/templating.rs index 9ae95ea..7976c65 100644 --- a/src/templating.rs +++ b/src/templating.rs @@ -344,7 +344,7 @@ pub fn format_document( // Initializing with 0 rather than relying on each ticket's `entry` call // is necessary for tickets that end up unused, because they wouldn't // call `entry` at all, and would report nothing. - for ticket in tickets.iter() { + for ticket in tickets { ticket_stats.insert(Rc::clone(&ticket.id), 0); } diff --git a/src/ticket_abstraction.rs b/src/ticket_abstraction.rs index f573ce7..47e2a68 100644 --- a/src/ticket_abstraction.rs +++ b/src/ticket_abstraction.rs @@ -214,7 +214,11 @@ impl IntoAbstract for Issue { // However, the project as a whole can be public or private, which affects the ticket visibility. // All projects are considered public unless you configure them in `JiraInstance::private_projects`. public: { - self.fields.security.is_none() && !config.jira.private_projects.contains(&self.fields.project.key) + self.fields.security.is_none() + && !config + .jira + .private_projects + .contains(&self.fields.project.key) }, references, }; diff --git a/src/tracker_access.rs b/src/tracker_access.rs index 838d445..be556bd 100644 --- a/src/tracker_access.rs +++ b/src/tracker_access.rs @@ -141,12 +141,12 @@ pub async fn unsorted_tickets( let mut annotated_tickets = Vec::new(); annotated_tickets.append(&mut into_annotated_tickets( plain_bugs, - &trackers, + trackers, &ref_signatures, )?); annotated_tickets.append(&mut into_annotated_tickets( plain_issues, - &trackers, + trackers, &ref_signatures, )?); @@ -311,7 +311,7 @@ async fn bugs_from_searches( ) -> Result, Bug)>> { let mut annotated_bugs: Vec<(Arc, Bug)> = Vec::new(); - for (search, query) in queries.iter() { + for (search, query) in queries { let mut bugs = bz_instance .search(search) // This enables the download concurrency: @@ -405,7 +405,7 @@ async fn issues_from_searches( ) -> Result, Issue)>> { let mut annotated_issues: Vec<(Arc, Issue)> = Vec::new(); - for (search, query) in queries.iter() { + for (search, query) in queries { let mut issues = jira_instance .search(search) // This enables the download concurrency: