Clean up custom code in `Vagrantfile`

Question

Clean up custom code in `Vagrantfile`

patcon opened this issue 12 years ago · 5 comments

Code Climate only evaluates proper modules and classes. The current code sucks at that. Needs to be cleaned up and moved into lib/ dir.

Original post

I'd like myself and everyone else to confront how bad my ruby can be, in the hopes that it will shame me into getting better.

Code Climate is a little like Travis CI, but for grading your coding style for each file via static analysis, and telling you when code gets better or worse with each commit

Just signed up: https://codeclimate.com/github/myplanetdigital/ariadne

Will add badge later

Answer 1 · 2012-09-16T00:55:21.000Z

Rats. Apparently it only scans app/ and lib/ dirs for files to grade, so unless I convert this into a gem, no dice.

Hey @brynary, I came across you being just generally awesome in this thread: ryanb/cancan#676

Any chance that codeclimate might be getting some like .travis.yml to customize things like setting file patterns?

Answer 2 · 2012-09-16T15:08:30.000Z

Hello!

Code Climate used to only do app/ and lib/, but now tries to find all Ruby files it doesn't think are config files or tests. The code must be defined inside classes or modules, however.

Does this get you what you need?

-Bryan

Answer 3 · 2012-09-16T17:30:19.000Z

That's totally useful! Thanks man! Will investigate why it's not picking up the very little code that meets that criteria :)

Answer 4 · 2012-09-18T22:35:11.000Z

TODO on this might as well involve removing all the hacky stuff in Vagrantfile, making it pretty, and putting it in lib/ (even if CodeClimate doesn't explicitly need it)

Answer 5 · 2012-09-20T19:16:43.000Z

Sahweet. It's picking stuff up now, so more incentive to do this :)
https://codeclimate.com/github/myplanetdigital/ariadne