Skip to content

Suppress non-json output - #33

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

weirdan wants to merge 1 commit into
ikappas:masterfrom
weirdan:run-phpcs-quietly

Conversation

@weirdan

@weirdan weirdan commented Feb 11, 2017

Copy link
Copy Markdown

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

ikappas commented Feb 11, 2017

Copy link
Copy Markdown
Owner

@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

weirdan commented Feb 11, 2017

Copy link
Copy Markdown
Author

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

@frekel

frekel commented Apr 19, 2017

Copy link
Copy Markdown

Hi,
Can you tell me when 0.8 is released?

Thanks,
Frank

@ntwb

ntwb commented Dec 16, 2017

Copy link
Copy Markdown

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