Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Mar 10, 2016

Format fails when directories contain spaces.

Windows 10:

Command failed: C:\Windows\system32\cmd.exe /s /c "yapf c:\tmp dir\test.py"
yapf: Input filenames did not match any python files

Ubuntu 14.04:

Command failed: /bin/sh -c autopep8 /vagrant/tmp dir/test.py
usage: autopep8 [-h] [--version] [-v] [-d] [-i] [--global-config filename]
[--ignore-local-config] [-r] [-j n] [-p n] [-a]
[--experimental] [--exclude globs] [--list-fixes]
[--ignore errors] [--select errors] [--max-line-length n]
[--line-range line line]
[files [files ...]]
autopep8: error: autopep8 only takes one filename as argument unless the "--in-place" or "--diff" args are used

Tested this branch in both OS's listed above.

DonJayamanne added a commit that referenced this pull request Mar 11, 2016
Added quotes around format-commands to prevent space errors.
@DonJayamanne DonJayamanne merged commit 4430718 into DonJayamanne:master Mar 11, 2016
@DonJayamanne
Copy link
Owner

Thanks, silly bug

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

Successfully merging this pull request may close these issues.

1 participant