Commit da185381 authored by nimrod's avatar nimrod
Browse files

- Corrected issue with shell-lint hook, the hook only checked the first 

script listed.
- Bumped patch version.
- Added pre-commit hooks to test the shell-lint script added.
parent cac4e3fa

.pre-commit-config.yaml

+7 −0
Changes for .pre-commit-config.yaml: 7 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -5,3 +5,10 @@ 
    -   id: check-added-large-files

    -   id: check-yaml

    -   id: check-merge-conflict

-   repo: https://www.shore.co.il/git/shell-pre-commit/

    sha: v0.5.0

    hooks:

    -   id: shell-lint

        files: shell-lint

    -   id: shellcheck

        files: shell-lint

.travis.yml

+7 −0
Changes for .travis.yml: 7 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -9,6 +9,13 @@ cache: 
  - directories:

      - $HOME/.pre-commit



addons:

  apt:

    sources:

      - sourceline: 'deb http://archive.ubuntu.com/ubuntu trusty-backports main universe multiverse'

    packages:

      - shellcheck



install:

  - pip install pre_commit

VERSION

+1 −1
Changes for VERSION: 1 added line, 1 removed line.
Original line number Diff line number Diff line 
0.5.0
 
 No newline at end of file
 
0.5.1
 
 No newline at end of file

hooks.yaml

+2 −3
Changes for hooks.yaml: 2 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -2,9 +2,8 @@ 
- id: shell-lint

  name: Lint shell scripts

  description: Run /bin/sh -en against shell scripts.

  language: system

  entry: /usr/bin/env

  args: ['-i', '/bin/sh', '-en']

  language: script

  entry: ./shell-lint

  files: &shell_scripts \.(sh|bash|ksh|zsh)$

- id: shellcheck

  name: shellcheck

shell-lint

0 → 100755
+6 −0
Changes for shell-lint: 6 added lines, 0 removed lines.
Original line number Diff line number Diff line 
#!/bin/sh

set -eu

for filename in "$@"

do

    /usr/bin/env -i /bin/sh "$filename"

done