atom-php-checkstyle icon indicating copy to clipboard operation
atom-php-checkstyle copied to clipboard

If the file has a large error report (circa 1500 errors), the report gets truncated and does not display all

Open gabriel403 opened this issue 11 years ago • 12 comments

As in this screenshot, it only shows for the first ~1k lines of code screenshot 2014-03-11 21 11 36

gabriel403 avatar Mar 11 '14 21:03 gabriel403

Currently unable to replicate this. I've just executed on a file which is 1866 lines long and the linter, phpcs and phpmd all showed errors on line 1865.

I assume you have run the command from the console window in the terminal and it is reporting errors?

benmatselby avatar Mar 12 '14 10:03 benmatselby

@benmatselby no just the plugin atm, don't know if it's the setup? screenshot 2014-03-12 10 51 00

gabriel403 avatar Mar 12 '14 10:03 gabriel403

What would the command be to run it from cli?

gabriel403 avatar Mar 12 '14 10:03 gabriel403

From inside of Atom, either do cmd+alt+i or "View -> Developer -> Toggle Developer Tools" and then click on the "Console" tab

This will then output the exact command that this plugin is running. Take that command and put it in your terminal and compare the results.

benmatselby avatar Mar 12 '14 12:03 benmatselby

screenshot 2014-03-12 12 14 05

gabriel403 avatar Mar 12 '14 12:03 gabriel403

OK, so wonder if it is the sheer amount of errors, rather than lines. How many errors do you have?

benmatselby avatar Mar 12 '14 12:03 benmatselby

Heh

"totals":{"errors":3106,"warnings":68}

gabriel403 avatar Mar 12 '14 12:03 gabriel403

Lemme prune the file down and see if I can find the magic error number

gabriel403 avatar Mar 12 '14 12:03 gabriel403

Install php-cs-fixer and hook that into the plugin then you can do cmd+k, cmd+f to run the fixer on the current file

benmatselby avatar Mar 12 '14 12:03 benmatselby

Updated title to reflect issue

benmatselby avatar Mar 12 '14 12:03 benmatselby

"totals":{"errors":1151,"warnings":40} anymore than that is what causes the gutter to not fully display errors

gabriel403 avatar Mar 12 '14 12:03 gabriel403

It is about the data we get back from child_process in stdout

benmatselby avatar Mar 12 '14 12:03 benmatselby

Closing this issue down, as it's been nearly a decade, and I'm going to archive the repo.

benmatselby avatar Jan 27 '23 20:01 benmatselby