swellaby / rusty-hook

git hook manager, geared toward Rust projects
MIT License
206 stars 11 forks source link

Better explanation for multiple commands in the Hooks section of the README #143

Closed Mastermindaxe closed 3 years ago

Mastermindaxe commented 3 years ago

Changes

Related Issues

Mastermindaxe commented 3 years ago

I'm not sure if we should just enhance the example with a command that uses && for command chaining. What do you think?

calebcartwright commented 3 years ago

I'm not sure if we should just enhance the example with a command that uses && for command chaining. What do you think?

I'd like to include this up above in the examples under the Configuration section.

Maybe we could change from this:


[hooks]
pre-commit = "cargo test"
prepare-commit-msg = "do-something && then-do-something-else"
# Array syntax for multiple hooks available in v0.12.0+
pre-push = ["cargo check", "cargo fmt -- --check"]
post-commit = "echo yay"

...
codecov[bot] commented 3 years ago

Codecov Report

Merging #143 (f3fb029) into master (e284e15) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #143   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          144       142    -2     
=========================================
- Hits           144       142    -2     
Impacted Files Coverage Δ
src/git.rs 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e284e15...f3fb029. Read the comment docs.