Skip to content

Commit

Permalink
refactor: use plural for vec options
Browse files Browse the repository at this point in the history
  • Loading branch information
RogueOneEcho committed Dec 16, 2024
1 parent 2d6e9d8 commit d032201
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions src/builder.rs
Original file line number Diff line number Diff line change
Expand Up @@ -38,17 +38,17 @@ impl LoggerBuilder {

#[must_use]
pub fn with_include_filter(mut self, include_filter: String) -> Self {
let mut filters = self.options.log_include_filter.unwrap_or_default();
let mut filters = self.options.log_include_filters.unwrap_or_default();
filters.push(include_filter);
self.options.log_include_filter = Some(filters);
self.options.log_include_filters = Some(filters);
self
}

#[must_use]
pub fn with_exclude_filter(mut self, exclude_filter: String) -> Self {
let mut filters = self.options.log_exclude_filter.unwrap_or_default();
let mut filters = self.options.log_exclude_filters.unwrap_or_default();
filters.push(exclude_filter);
self.options.log_exclude_filter = Some(filters);
self.options.log_exclude_filters = Some(filters);
self
}

Expand Down
4 changes: 2 additions & 2 deletions src/logger.rs
Original file line number Diff line number Diff line change
Expand Up @@ -51,14 +51,14 @@ impl Logger {
}

fn exclude_by_target(&self, target: &str) -> bool {
if let Some(exclude_filters) = self.options.log_exclude_filter.clone() {
if let Some(exclude_filters) = self.options.log_exclude_filters.clone() {
for filter in exclude_filters {
if target.starts_with(&filter) {
return true;
}
}
}
if let Some(mut include_filters) = self.options.log_include_filter.clone() {
if let Some(mut include_filters) = self.options.log_include_filters.clone() {
include_filters.push(PACKAGE_NAME.to_owned());
for filter in include_filters {
if !target.starts_with(&filter) {
Expand Down
4 changes: 2 additions & 2 deletions src/options.rs
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ pub struct LoggerOptions {
pub log_time_format: Option<TimeFormat>,

/// Include only logs from specific packages
pub log_include_filter: Option<Vec<String>>,
pub log_include_filters: Option<Vec<String>>,

/// Exclude logs from specific packages
pub log_exclude_filter: Option<Vec<String>>,
pub log_exclude_filters: Option<Vec<String>>,
}

0 comments on commit d032201

Please sign in to comment.