Merge pull request #504 from pre-commit/use_version_everywhere

Use our VERSION constant moar
This commit is contained in:
Anthony Sottile 2017-03-06 13:40:52 -08:00 committed by GitHub
commit 71bbdb5b4c
2 changed files with 4 additions and 12 deletions

View file

@ -7,9 +7,9 @@ import re
import jsonschema
import jsonschema.exceptions
import pkg_resources
import yaml
import pre_commit.constants as C
from pre_commit import output
from pre_commit.jsonschema_extensions import apply_defaults
@ -72,11 +72,7 @@ def get_run_function(filenames_help, validate_strategy, exception_cls):
parser = argparse.ArgumentParser()
parser.add_argument('filenames', nargs='*', help=filenames_help)
parser.add_argument(
'-V', '--version',
action='version',
version='%(prog)s {}'.format(
pkg_resources.get_distribution('pre-commit').version
)
'-V', '--version', action='version', version=C.VERSION,
)
args = parser.parse_args(argv)

View file

@ -8,10 +8,9 @@ import re
import shutil
import mock
import pkg_resources
import pytest
from pre_commit import constants as C
import pre_commit.constants as C
from pre_commit import five
from pre_commit import parse_shebang
from pre_commit.clientlib.validate_manifest import load_manifest
@ -721,10 +720,7 @@ def test_too_new_version(tempdir_factory, store, fake_log_handler):
)
@pytest.mark.parametrize(
'version',
('0.1.0', pkg_resources.get_distribution('pre-commit').version),
)
@pytest.mark.parametrize('version', ('0.1.0', C.VERSION))
def test_versions_ok(tempdir_factory, store, version):
path = make_repo(tempdir_factory, 'script_hooks_repo')
with modify_manifest(path) as manifest: