refactor(config): Clarify config is not file-specific

This is prep for the config being reused in other contexts, like commit
messages.
This commit is contained in:
Ed Page committed 2021-03-29 20:27:12 -05:00
1 parent 4bbc59facf
commit f402d3ee77
2 files changed
+11 -11

No files matched your search

+1 -1
View File
@@ -129,7 +129,7 @@ pub(crate) struct FileArgs {
pub(crate) locale: Option<config::Locale>,
}
impl config::FileSource for FileArgs {
impl config::EngineSource for FileArgs {
fn binary(&self) -> Option<bool> {
match (self.binary, self.no_binary) {
(true, false) => Some(true),
+10 -10
View File
@@ -5,7 +5,7 @@ pub trait ConfigSource {
None
}
fn default(&self) -> Option<&dyn FileSource> {
fn default(&self) -> Option<&dyn EngineSource> {
None
}
}
@@ -42,7 +42,7 @@ pub trait WalkSource {
}
}
pub trait FileSource {
pub trait EngineSource {
/// Check binary files.
fn binary(&self) -> Option<bool> {
None
@@ -113,7 +113,7 @@ pub trait DictSource {
#[serde(rename_all = "kebab-case")]
pub struct Config {
pub files: Walk,
pub default: FileConfig,
pub default: EngineConfig,
}
impl Config {
@@ -130,7 +130,7 @@ impl Config {
pub fn from_defaults() -> Self {
Self {
files: Walk::from_defaults(),
default: FileConfig::from_defaults(),
default: EngineConfig::from_defaults(),
}
}
@@ -157,7 +157,7 @@ impl ConfigSource for Config {
Some(&self.files)
}
fn default(&self) -> Option<&dyn FileSource> {
fn default(&self) -> Option<&dyn EngineSource> {
Some(&self.default)
}
}
@@ -266,7 +266,7 @@ impl WalkSource for Walk {
#[derive(Debug, Clone, Default, serde::Serialize, serde::Deserialize)]
#[serde(deny_unknown_fields, default)]
#[serde(rename_all = "kebab-case")]
pub struct FileConfig {
pub struct EngineConfig {
pub binary: Option<bool>,
pub check_filename: Option<bool>,
pub check_file: Option<bool>,
@@ -276,10 +276,10 @@ pub struct FileConfig {
pub dict: Option<DictConfig>,
}
impl FileConfig {
impl EngineConfig {
pub fn from_defaults() -> Self {
let empty = Self::default();
FileConfig {
EngineConfig {
binary: Some(empty.binary()),
check_filename: Some(empty.check_filename()),
check_file: Some(empty.check_file()),
@@ -292,7 +292,7 @@ impl FileConfig {
}
}
pub fn update(&mut self, source: &dyn FileSource) {
pub fn update(&mut self, source: &dyn EngineSource) {
if let Some(source) = source.binary() {
self.binary = Some(source);
}
@@ -333,7 +333,7 @@ impl FileConfig {
}
}
impl FileSource for FileConfig {
impl EngineSource for EngineConfig {
fn binary(&self) -> Option<bool> {
self.binary
}