Rubocop warning fixes

[guilhem]

From https://github.com/redguide/nodejs/blob/master/.rubocop.yml

[fonsecas72]

This does not seem to affect anything, Am I wrong?
But I would like to add consistency between projects.
For what I'm used to see, I would only change the vendor's igore to

 - 'vendor/**/*'

[guilhem]

@fonsecas72 problems seems to appear here: https://travis-ci.org/redguide/locales/jobs/53025883

[fonsecas72]

It looks like only the vendor's dir exclusion is deprecated and that's why I would not change the other files inclusions because I usually see file inclusion (edited) as they were.
I'de say, let's choose one way or another and apply it to all cookbooks.

[guilhem]

@fonsecas72 include as it's done in this patch is cool for test-cookbooks that can be in test directory.
But it's obviously not a major things.

Let me know what @redguide/locales are thinking and I will change my commit :)

[guilhem]

oh and also @fonsecas72 there is no problem to me if 2 teams of maintainers don't want same syntax or behavior.
What I only want is discussions, examples and contributions between each other :) (and I'm sure the better will won at the end by "success story" ;))

"big bang" changes have too many chances to fail and to freeze any new way of thinking ^^

[BarthV]

Using test cookbooks can be very useful in the future, if we feel the need to add some QA/tests.
👍 for this PR syntax

[fonsecas72]

Using test cookbooks can be very useful in the future

I don't know how this can become handy yet, so 👍 to learn. ;-)