Skip to content

Commit

Permalink
Added ShellCheck as requirement and fixed tests for a new version
Browse files Browse the repository at this point in the history
  • Loading branch information
vint21h committed Jul 26, 2022
1 parent b9841cf commit f89bd09
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@ test_suite = tests
scripts =
pre_commit_config_shellcheck.py
install_requires =
pyyaml==6.0
pyyaml>=6.0
shellcheck-py>=0.8.0.4

[options.package_data]
pre-commit-config-shellcheck = README.rst, MIT-LICENSE, CONTRIBUTORS
Expand Down
4 changes: 2 additions & 2 deletions tests/test_pre_commit_config_shellcheck.py
Original file line number Diff line number Diff line change
Expand Up @@ -367,13 +367,13 @@ def test_pre_commit_config_shellcheck___check_entries(
expected = """
In entry "removestar" on line 17:
removestar -i ${NAME}
^-----^ SC2086: Double quote to prevent globbing and word splitting.
^-----^ SC2086 (info): Double quote to prevent globbing and word splitting.
Did you mean: \nremovestar -i "${NAME}"
For more information:
https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...
"""
""" # noqa: E501
assert captured.out == expected


Expand Down

0 comments on commit f89bd09

Please sign in to comment.