Compare commits

..

No commits in common. "651699c40aeff48c6af797c7182bcd3f83171d31" and "4ba723d5cc0709bb31f82e7143fc2b58f7eee3d3" have entirely different histories.

9 changed files with 3 additions and 47 deletions

1
Cargo.lock generated
View File

@ -453,7 +453,6 @@ dependencies = [
"assert_cmd",
"clap",
"color-eyre",
"globset",
"hooked-config",
"insta",
"owo-colors",

View File

@ -16,7 +16,6 @@ path = "source/main.rs"
[dependencies]
color-eyre = "0.6.2"
globset = "0.4.9"
owo-colors = "3.5.0"
subprocess = "0.2.9"
supports-color = "1.3.0"

View File

@ -4,7 +4,6 @@ use std::{io::Read, process::exit};
use {
color_eyre::{eyre::eyre, Result},
globset::{Glob, GlobSetBuilder},
hooked_config::{Config, ExitAction},
owo_colors::{OwoColorize, Style},
subprocess::{Exec, Redirection},
@ -15,17 +14,16 @@ use crate::utilities::plural;
/// The `run` subcommand.
pub fn hooked_run(config: Config, hook_type: String) -> Result<()> {
let (success_style, warn_style, error_style, skipped_style) =
let (success_style, warn_style, error_style) =
if let Some(_support) = supports_color::on(Stream::Stdout) {
let shared_style = Style::new().bold();
(
shared_style.green(),
shared_style.yellow(),
shared_style.red(),
shared_style.blue(),
)
} else {
(Style::new(), Style::new(), Style::new(), Style::new())
(Style::new(), Style::new(), Style::new())
};
if hook_type == "pre-commit" {
@ -36,35 +34,9 @@ pub fn hooked_run(config: Config, hook_type: String) -> Result<()> {
plural(hook_count, "hook", None)
);
'hook_loop: for hook in config.pre_commit {
for hook in config.pre_commit {
let hook_name = hook.name.unwrap_or_else(|| "Unnamed Hook".to_string());
if !hook.git_staged.is_empty() {
let globs = {
let mut builder = GlobSetBuilder::new();
for glob in hook.git_staged {
builder.add(Glob::new(&glob)?);
}
builder.build()?
};
let staged_files = Exec::cmd("git")
.args(&["diff", "--name-only", "--cached"])
.capture()?
.stdout_str();
for line in staged_files.lines() {
if globs.is_match(line) {
println!(
"\t{} {}",
"".style(skipped_style),
hook_name.style(skipped_style)
);
continue 'hook_loop;
}
}
}
let command = match (hook.task.command, hook.task.script) {
(Some(command), _) => Ok(Exec::shell(command)),

View File

@ -8,10 +8,6 @@ use crate::{ExitAction, Task};
#[derive(Debug, Deserialize, Serialize)]
#[serde(deny_unknown_fields)]
pub struct PreCommit {
/// List of globs to check against staged files.
#[serde(default)]
pub git_staged: Vec<String>,
/// Display name for this hook.
pub name: Option<String>,

View File

@ -5,7 +5,6 @@ template = "test.sh"
[[pre_commit]]
name = "Pre Commit 1"
command = "exit 0"
git_staged = ["*.txt"]
on_failure = "continue"
[[pre_commit]]

View File

@ -8,7 +8,6 @@ use insta::assert_snapshot;
#[test]
fn test_serialize() {
let pre_commit_command = PreCommit {
git_staged: vec!["*.txt".to_string()],
name: Some("Command Test".to_string()),
on_failure: ExitAction::Continue,
task: Task {
@ -18,7 +17,6 @@ fn test_serialize() {
};
let pre_commit_script = PreCommit {
git_staged: vec![],
name: Some("Script Test".to_string()),
on_failure: ExitAction::Stop,
task: Task {

View File

@ -10,7 +10,6 @@ Config {
},
pre_commit: [
PreCommit {
git_staged: [],
name: None,
on_failure: Stop,
task: Task {

View File

@ -12,9 +12,6 @@ Config {
},
pre_commit: [
PreCommit {
git_staged: [
"*.txt",
],
name: Some(
"Pre Commit 1",
),
@ -27,7 +24,6 @@ Config {
},
},
PreCommit {
git_staged: [],
name: Some(
"Pre Commit 2",
),

View File

@ -7,13 +7,11 @@ config = 'Hooked.toml'
directory = 'hooks'
[[pre_commit]]
git_staged = ['*.txt']
name = 'Command Test'
on_failure = 'continue'
command = 'exit 0'
[[pre_commit]]
git_staged = []
name = 'Script Test'
on_failure = 'stop'
script = 'test.sh'