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.

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

Use different seperator for application directories #60

Closed
timonbimon opened this issue Dec 16, 2018 · 2 comments
Closed

Use different seperator for application directories #60

timonbimon opened this issue Dec 16, 2018 · 2 comments

Comments

@timonbimon
Copy link

Hi again :)

Is it possible to use a seperator other than ':' for the application directories.
The colon does not seem to play well with the .pre-commit-config.yaml

For the following .-pre-commit-config.yaml

repos:
-   repo: https://github.com/asottile/reorder_python_imports
    rev: v1.3.4
    hooks:
    - id: reorder-python-imports
      args: [--application-directories=.:src]
      language_version: python3.7

I get the following error:

An error has occurred: InvalidConfigError: 
==> File /Users/timon/code/luminovo/project/.pre-commit-config.yaml
=====> while scanning a plain scalar
  in "<unicode string>", line 6, column 14:
          args: [--application-directories=.:src]
                 ^
found unexpected ':'
  in "<unicode string>", line 6, column 41:
     ... gs: [--application-directories=.:src]
                                         ^
Please check http://pyyaml.org/wiki/YAMLColonInFlowContext for details.
@asottile
Copy link
Owner

You should quote the string, see for example pytest's config

@timonbimon
Copy link
Author

awesome thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants