Skip/remove swiftlint task #149
Description
Hello @ashfurrow . First of all, nice plugin et very nice huge work!
I would like to know if you plan to make the run of swiftlint not mandatory?
I ask the question because i really think that when Orta created Danger (maybe iam wrong), he designed it to be a tool that take an output, work on it and send it as comment on Github PR.
Because of the design of the plugin, we have to fill the binary path, then swiftlint is running. But its running twice because juste before, i have setup swiftlint as a buildphase in my xcodeproj. So to avoid it, i have to write something like "if i am not running on CI, ./Pods/swiftlint/swiftlint". But that mean i create a dependency on my local build and CI build.
I think this plugin should just have parameter to have a path/to/swiftlint_report and a parse method, maybe some options to filter and format the report and that's all.
Isn't it?
You can have a look on Orta's danger-junit plugin here : https://github.com/orta/danger-junit/ and on the image in attachement. I know that you are a big contributor and i just want to have your point of view on this.
I understand it could be a potential breaking change and I propose my help to create a new parameter that will prevent raising an exception if the binary is not in the path, and it will just parse the report path and thats it. If you agree, i can work on it :)
