Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(fix): Use venv created by pre-commit for validate config #3166

Merged
merged 3 commits into from Sep 23, 2022
Merged

(fix): Use venv created by pre-commit for validate config #3166

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b3a578f
Create venv for pre-commit validate config
enochtangg Sep 21, 2022
3f38084
Add files to run on
enochtangg Sep 21, 2022
6035d1f
Fix files glob to regex
enochtangg Sep 22, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
View file
Open in desktop
Expand Up @@ -41,8 +41,8 @@ repos:
hooks:
- id: validate-configs-syntax
name: validate-configs-syntax
entry: python3 ./snuba/validate_configs.py
language: system
pass_filenames: false
entry: python3 -m snuba.validate_configs
language: python
additional_dependencies: [ 'jsonschema', 'pyyaml' ]
enochtangg marked this conversation as resolved.
Show resolved Hide resolved
default_language_version:
python: python3.8