rust-test: init

Author: domodwyer

.pre-commit-hooks.yaml

@@ -1,32 +1,3 @@
-- id: go-test
-  name: 'Go tests'
-  entry: run-go-test.sh
-  types: [go]
-  language: 'script'
-  pass_filenames: false
-  minimum_pre_commit_version: 0.15.0
-  description: "Runs `go test ./...` at the repo root"
-
-- id: goimports
-  name: 'Check Go imports'
-  entry: run-go-imports.sh
-  types: [go]
-  language: 'script'
-  pass_filenames: true
-  minimum_pre_commit_version: 0.15.0
-  description: "Runs `goimports` on changed files"
-  require_serial: true
-  exclude: 'vendor/'
-
-- id: go-dep-check
-  name: 'dep-check'
-  entry: run-dep-check.sh
-  types: [go]
-  language: 'script'
-  pass_filenames: false
-  minimum_pre_commit_version: 0.15.0
-  description: "Runs `dep check`, ensuring all imports are vendored"
-
 - id: todo-tagged-jira
   name: 'TODOs have JIRA tags'
   entry: todo-jira-check.py
@@ -47,19 +18,48 @@
   description: 'Prints any TODOs tagged with the ticket reference in the current branch name'
 
 - id: branch-name-check
-  name: 'Checks branch name obeys the standardised format'
+  name: 'Branch name obeys the standardised format'
   entry: run-branch-name-check.sh
   always_run: true
   pass_filenames: false
   language: 'script'
   minimum_pre_commit_version: 0.15.0
   description: 'Checks branch names obeys the following regex'
 
+- id: go-test
+  name: 'Go tests'
+  entry: run-go-test.sh
+  types: [go]
+  language: 'script'
+  pass_filenames: false
+  minimum_pre_commit_version: 0.15.0
+  description: "Runs `go test ./...` at the repo root"
+
+- id: go-goimports
+  name: 'Check Go imports'
+  entry: run-go-imports.sh
+  types: [go]
+  language: 'script'
+  pass_filenames: true
+  minimum_pre_commit_version: 0.15.0
+  description: "Runs `goimports` on changed files"
+  require_serial: true
+  exclude: 'vendor/'
+
+- id: go-dep-check
+  name: 'Dependencies are vendored (dep)'
+  entry: run-dep-check.sh
+  types: [go]
+  language: 'script'
+  pass_filenames: false
+  minimum_pre_commit_version: 0.15.0
+  description: "Runs `dep check`, ensuring all imports are vendored"
+
 # A copy of
 # https://github.com/golangci/golangci-lint/commit/09677d574ea6cd05141022aa90b88b6598bfa1a1
 # without --fix forced on users.
-- id: golangci-lint
-  name: golangci-lint
+- id: go-golangci-lint
+  name: 'Run golangci-lint'
   description: Static analysis for Go code
   entry: golangci-lint run
   types: [go]
@@ -75,6 +75,15 @@
   minimum_pre_commit_version: 0.15.0
   description: 'Run clippy lints against rust code'
 
+- id: rust-test
+  name: 'Cargo tests'
+  entry: run-rust-tests.sh
+  language: 'script'
+  files: \.rs$
+  pass_filenames: false
+  minimum_pre_commit_version: 0.15.0
+  description: 'Run cargo test in the repo root'
+
 - id: r-stylr
   name: 'Format with stylr'
   entry: run-r-stylr.R

README.md

@@ -1,18 +1,19 @@
 # pre-commit
 
-This is a set of [`pre-commit`] hooks for (primarily Go) development.
+This is a set of [`pre-commit`] hooks for development across multiple languages.
 
 Hooks:
 * `todo-tagged-jira`: ensure all TODO comments reference a JIRA ticket
 * `todo-branch-tags`: show all TODOs tagged with the ticket reference in the branch name
 * `branch-name-check`: checks branch names adhere to the regex `^(feature|bugfix|release|hotfix)\/.+`
 * `go-test`: runs `go test ./...` at the repo root
 * `go-dep-check`: ensure all your 3rd party Go packages are vendored (see [dep])
-* `goimports`: ensure all the Go imports are included and ordered
-* `golangci-lint`: a copy of the [official lint
+* `go-goimports`: ensure all the Go imports are included and ordered
+* `go-golangci-lint`: a copy of the [official lint
   config](https://github.com/golangci/golangci-lint/commit/09677d574ea6cd05141022aa90b88b6598bfa1a1)
   without forcing the `--fix` argument
 * `rust-clippy`: runs `cargo clippy` lints in the repo root
+* `rust-test`: runs `cargo test` at the repo root, includes all targets/features/examples/benches
 * `r-stylr`: runs [`stylr`] to format R code
 * `r-lintr`: static analysis of R code with [`lintr`]
 
@@ -40,7 +41,7 @@ repos:
         exclude: \.pb.go$                  # Ignore generated protobuf files
         args: ["-timeout=30s"]             # Set a deadline (fast tests == happy developers)
 
-      - id: goimports
+      - id: go-goimports
         stages: [commit, push, manual]
         types: [go]
         exclude: \.pb.go$                  # Ignore generated protobuf files
@@ -50,13 +51,20 @@ repos:
         stages: [push, manual]
         types: [go]
 
-      - id: golangci-lint
+      - id: go-golangci-lint
         args: [--new-from-rev=origin/master]
         stages: [commit, push]
         types: [go]
         exclude: \.pb.go$
 
       - id: rust-clippy
+        #args: [                           # Optionally override default configured lints
+        #  "-D rust_2018_idioms",
+        #  "-D missing_docs",
+        #]
+        stages: [commit, push]
+      
+      - id: rust-test
         stages: [commit, push]
 
       - id: r-stylr

run-rust-tests.sh

@@ -0,0 +1,5 @@
+#!/usr/bin/env bash
+
+set -euo pipefail
+
+cargo test --all-features --all --bins --tests --examples --benches --all-targets "$@"