-
Notifications
You must be signed in to change notification settings - Fork 6
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 Cake.Issues.Recipe #12
Comments
@pascalberger sounds like a plan to me. What format is currently being used by the ESLint Addin? Can't we just use that? |
It currently supports JSON formatter |
Oh right, I see (I think)... You are saying that the ESLint addin supports JOSN log files, but currently there isn't a way within the Cake.Issues.Recipe to bring those in. Is that correct? |
Yes, but it's not a big deal to add ESLint support to Cake.Issues.Recipe, that's quickly done. I think the biggest task to solve here is how to call the linting in a way that the recipe script knows the output path afterwards. |
@pascalberger I know you like a challenge 😛 |
Cake.Issues.Recipe 0.4.4 has been released with support for ESLint json formatter output |
@pascalberger woot! Are you in a position to add this into the Recipe here? |
We could add Cake.Issues.Recipe for handling of issues reported by ESLint. Requires cake-contrib/Cake.Issues.Recipe#3 to be implemented first.
The text was updated successfully, but these errors were encountered: