Merge pull request #255 from epage/ignore

feat(ignore): Typos=specific ignores
This commit is contained in:
Ed Page 2021-05-21 09:23:57 -05:00 committed by GitHub
commit 5eb4665c83
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 25 additions and 0 deletions

View file

@ -14,6 +14,8 @@ Configuration is read from the following (in precedence order)
| Field | Argument | Format | Description | | Field | Argument | Format | Description |
|------------------------|-------------------|--------|-------------| |------------------------|-------------------|--------|-------------|
| files.binary | --binary | bool | Check binary files as text |
| files.extend-exclude | --exclude | list of strings | Typos-specific ignore globs (gitignore syntax) |
| files.ignore-hidden | --hidden | bool | Skip hidden files and directories. | | files.ignore-hidden | --hidden | bool | Skip hidden files and directories. |
| files.ignore-files | --ignore | bool | Respect ignore files. | | files.ignore-files | --ignore | bool | Respect ignore files. |
| files.ignore-dot | --ignore-dot | bool | Respect .ignore files. | | files.ignore-dot | --ignore-dot | bool | Respect .ignore files. |

View file

@ -200,6 +200,10 @@ impl ConfigArgs {
#[derive(Debug, StructOpt)] #[derive(Debug, StructOpt)]
#[structopt(rename_all = "kebab-case")] #[structopt(rename_all = "kebab-case")]
pub(crate) struct WalkArgs { pub(crate) struct WalkArgs {
#[structopt(long)]
/// Ignore all files & directories matching the pattern.
exclude: Vec<String>,
#[structopt(long, overrides_with("no-hidden"))] #[structopt(long, overrides_with("no-hidden"))]
/// Search hidden files and directories. /// Search hidden files and directories.
hidden: bool, hidden: bool,
@ -240,6 +244,7 @@ pub(crate) struct WalkArgs {
impl WalkArgs { impl WalkArgs {
pub fn to_config(&self) -> config::Walk { pub fn to_config(&self) -> config::Walk {
config::Walk { config::Walk {
extend_exclude: self.exclude.clone(),
ignore_hidden: self.ignore_hidden(), ignore_hidden: self.ignore_hidden(),
ignore_files: self.ignore_files(), ignore_files: self.ignore_files(),
ignore_dot: self.ignore_dot(), ignore_dot: self.ignore_dot(),

View file

@ -196,6 +196,16 @@ fn run_checks(
.git_ignore(walk_policy.ignore_vcs()) .git_ignore(walk_policy.ignore_vcs())
.git_exclude(walk_policy.ignore_vcs()) .git_exclude(walk_policy.ignore_vcs())
.parents(walk_policy.ignore_parent()); .parents(walk_policy.ignore_parent());
if !walk_policy.extend_exclude.is_empty() {
let mut overrides = ignore::overrides::OverrideBuilder::new(".");
for pattern in walk_policy.extend_exclude.iter() {
overrides
.add(&format!("!{}", pattern))
.with_code(proc_exit::Code::CONFIG_ERR)?;
}
let overrides = overrides.build().with_code(proc_exit::Code::CONFIG_ERR)?;
walk.overrides(overrides);
}
// HACK: Diff doesn't handle mixing content // HACK: Diff doesn't handle mixing content
let output_reporter = if args.diff { let output_reporter = if args.diff {

View file

@ -61,6 +61,7 @@ impl Config {
#[serde(deny_unknown_fields, default)] #[serde(deny_unknown_fields, default)]
#[serde(rename_all = "kebab-case")] #[serde(rename_all = "kebab-case")]
pub struct Walk { pub struct Walk {
pub extend_exclude: Vec<String>,
/// Skip hidden files and directories. /// Skip hidden files and directories.
pub ignore_hidden: Option<bool>, pub ignore_hidden: Option<bool>,
/// Respect ignore files. /// Respect ignore files.
@ -79,6 +80,7 @@ impl Walk {
pub fn from_defaults() -> Self { pub fn from_defaults() -> Self {
let empty = Self::default(); let empty = Self::default();
Self { Self {
extend_exclude: empty.extend_exclude.clone(),
ignore_hidden: Some(empty.ignore_hidden()), ignore_hidden: Some(empty.ignore_hidden()),
ignore_files: Some(true), ignore_files: Some(true),
ignore_dot: Some(empty.ignore_dot()), ignore_dot: Some(empty.ignore_dot()),
@ -89,6 +91,8 @@ impl Walk {
} }
pub fn update(&mut self, source: &Walk) { pub fn update(&mut self, source: &Walk) {
self.extend_exclude
.extend(source.extend_exclude.iter().cloned());
if let Some(source) = source.ignore_hidden { if let Some(source) = source.ignore_hidden {
self.ignore_hidden = Some(source); self.ignore_hidden = Some(source);
} }
@ -114,6 +118,10 @@ impl Walk {
} }
} }
pub fn extend_exclude(&self) -> &[String] {
&self.extend_exclude
}
pub fn ignore_hidden(&self) -> bool { pub fn ignore_hidden(&self) -> bool {
self.ignore_hidden.unwrap_or(true) self.ignore_hidden.unwrap_or(true)
} }