Skip to content
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

Rubocop plugin #120

Closed
wants to merge 22 commits into from
Closed

Rubocop plugin #120

wants to merge 22 commits into from

Conversation

chazzly
Copy link
Contributor

@chazzly chazzly commented Apr 23, 2014

Created a plug-in to run Rubocop against a cookbook before uploading, based completely on the Foodcritic plug-in.

There's more I would like to do with it if I can figure out the rubocop API more, but this does the basic - runs rubocop, fails if 'epic_fail' is set to true.

I'm still learning git (along with Chef & Ruby), so forgive me if I've done the branching / pull-request wrong somehow.

@chazzly
Copy link
Contributor Author

chazzly commented Apr 29, 2014

I figured out how to pass the command line options to rubocop and added them the spork tags.

@jonlives jonlives added this to the Version 1.4.0 milestone Jul 21, 2014
@jonlives
Copy link
Owner

Added (belatedly) to milestone for 1.4.0 release

@jonlives
Copy link
Owner

@chazzly apologies for the delay on getting this merged - could you please add the relevant sample config etc to README.md in addition to your plugin doc please? Once that's done I'll get this merged and out in the next release this week

@chazzly
Copy link
Contributor Author

chazzly commented Aug 19, 2014

Added to the README.md Let me know if that's not what you're after.

@jonlives
Copy link
Owner

@chazzly thanks for that! I'll prob move the verbose section into the plugin doc (existing plugins are only documented in README.md in the config example), but that's easy enough. Will get this merged tomorrow most likely.

@jonlives
Copy link
Owner

Merged and released in 1.4.0

@jonlives jonlives closed this Aug 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants