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

Improve console output formatting. #967

Closed
ADmad opened this issue Sep 3, 2018 · 3 comments
Closed

Improve console output formatting. #967

ADmad opened this issue Sep 3, 2018 · 3 comments
Assignees

Comments

@ADmad
Copy link
Contributor

ADmad commented Sep 3, 2018

The current console error/info output feels messy and not very readable (even with code snippets turned off) if you have largish number of errors.

It would be nice if it was possible to get a compact output with errors groups by file like phpstan or somewhat tabular display like phpcs.

@ADmad ADmad changed the title Improve console output format. Improve console output formatting. Sep 3, 2018
@muglug
Copy link
Collaborator

muglug commented Sep 12, 2018

If you provide a format you'd like I could implement it as an option in --output-format

@weirdan
Copy link
Collaborator

weirdan commented Sep 14, 2018

What both of them have in common is that they output errors grouped by file (psalm outputs ungrouped list), as well as highlighting the file name (psalm doesn't highlight it). However default psalm output includes snippets, which is also quite useful, so one line per issue is not achievable unless snippets are dropped. But grouping could be done.

If you're looking into the output format, it would also be nice to support terminal hyperlinks

@ADmad
Copy link
Contributor Author

ADmad commented Sep 14, 2018

I would like to have an out which resembles the following:

FILE: /path/to/code/myfile.php
--------------------------------------------------------------------------------
SEVERITY | LINE | ISSUE                   | DESCRIPTION
--------------------------------------------------------------------------------
ERROR    |    2 | InvalidArgument         | Lorem ipsum
INFO     |  120 | NullableReturnStatement | Lorem ipsum
--------------------------------------------------------------------------------

Columns 1 and 2 column be swapped to keep it similar to phpcs.

@erunion erunion self-assigned this Dec 6, 2018
erunion pushed a commit that referenced this issue Dec 9, 2018
The new compact format generates your report within a table
structure that is similar to PHPCS.

Resolves #967
muglug pushed a commit that referenced this issue Dec 9, 2018
The new compact format generates your report within a table
structure that is similar to PHPCS.

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

No branches or pull requests

4 participants