benmatselby / atom-php-checkstyle

✅ [DEPRECATED] Checkstyle reporter for PHP, utilising phpcs, phpmd, php linter and php-cs-fixer
MIT License
21 stars 5 forks source link

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

Closed gabriel403 closed 1 year ago

gabriel403 commented 10 years ago

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

benmatselby commented 10 years ago

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?

gabriel403 commented 10 years ago

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

gabriel403 commented 10 years ago

What would the command be to run it from cli?

benmatselby commented 10 years ago

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.

gabriel403 commented 10 years ago

screenshot 2014-03-12 12 14 05

benmatselby commented 10 years ago

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

gabriel403 commented 10 years ago

Heh

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

gabriel403 commented 10 years ago

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

benmatselby commented 10 years ago

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 commented 10 years ago

Updated title to reflect issue

gabriel403 commented 10 years ago

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

benmatselby commented 10 years ago

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

benmatselby commented 1 year ago

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