Skip to content

Suppress non-json output#33

Closed
weirdan wants to merge 1 commit intoikappas:masterfrom
weirdan:run-phpcs-quietly
Closed

Suppress non-json output#33
weirdan wants to merge 1 commit intoikappas:masterfrom
weirdan:run-phpcs-quietly

Conversation

@weirdan
Copy link
Copy Markdown

@weirdan weirdan commented Feb 11, 2017

When a standard specifies <arg value="p"/> (progress) option linter
breaks since it expects json output, and PHPCS spits out dots for
passing files and 'E' for errors.

Adding -q option suppresses progres output.

Maybe related: #32
See the same issue in PHPCS linter for atom: AtomLinter/linter-phpcs#95

When a standard specifies `<arg value="p"/>` (progress) option linter
breaks since it expects json output, and PHPCS spits out dots for
passing files and 'E' for errors.

Adding `-q` option suppresses progres output.

Maybe related: ikappas#32
See the same issue in PHPCS linter for atom: AtomLinter/linter-phpcs#95
@ikappas
Copy link
Copy Markdown
Owner

ikappas commented Feb 11, 2017

@weirdan Thank you for reporting this issue! This has been merged into develop and will be available in vscode-phpcs 0.8

@ikappas ikappas closed this Feb 11, 2017
@weirdan
Copy link
Copy Markdown
Author

weirdan commented Feb 11, 2017

Oh, I should have made the pull request against develop.

@frekel
Copy link
Copy Markdown

frekel commented Apr 19, 2017

Hi,
Can you tell me when 0.8 is released?

Thanks,
Frank

@ntwb
Copy link
Copy Markdown

ntwb commented Dec 16, 2017

Any chance of a new release soon please @ikappas ? I see a handful of 1.x betas, will 1.x be soon?

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.

4 participants