-
Notifications
You must be signed in to change notification settings - Fork 17
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
Add suport for php-cs-fixer #94
Comments
Like cs2pr https://github.com/staabm/annotate-pull-request-from-checkstyle, |
I noticed, php-cs-fixer would not print Line Number. So, currently we could not support php-cs-fixer simply via
line number could be get only with
|
Hi @sasezaki thanks for the feature request and for links to research you've already done. I'm not sure I'll be able to add this due to issue that php-cs-fixer does not supply line number. We might have to wait until there is support from the php-cs-fixer before progressing. I'll take a bit more of a look at this and the diffs later this evening (I'm on UK time). |
Without line numbers it is not going to be possible to implement this. The relevant issue is PHP-CS-Fixer/PHP-CS-Fixer#3601 If a PR for this, or a similar PR that adds line numbers to an output gets merged then I'll happily create a ResultsParser for the format. For now I'll close this issue. |
Hi.
As feature request, I want php-cs-fixer support.
https://github.com/FriendsOfPHP/PHP-CS-Fixer
The text was updated successfully, but these errors were encountered: