textmate2-rubocop icon indicating copy to clipboard operation
textmate2-rubocop copied to clipboard

Published 20 hours ago • verified publisher icon mrdougal

Don't bundle rubocop; s/Rubocop/RuboCop/

Open saizai opened this issue 10 years ago • 5 comments

  1. Bundled version is out of date. Don't include it in the textmate bundle; use the gem version.
  2. The current version uses RuboCop:: not Rubocop::.

saizai avatar Jul 14 '14 09:07 saizai

Granted the bundled version of RuboCop is out of date. I'm confused with your statement. "Don't include it in the textmate bundle; use the gem version" are you saying i shouldn't bundle at all? Or is the bundle not using a version of RuboCop that you have installed outside of the plugin?

"The current version uses RuboCop:: not Rubocop::." - sorry i don't understand what you mean in this statement. Can you elaborate?

Thanks

mrdougal avatar Jul 15 '14 02:07 mrdougal

Yes, the bundle shouldn't include the gem, just a ’require’.

For #2 I meant that literally. The case changed.

saizai avatar Jul 15 '14 07:07 saizai

Any plans to fix this?

brianviveiros avatar Oct 24 '15 02:10 brianviveiros

Hello To be honest, not right now. In part because i'm not writing much ruby at the moment, and i couldn't get the code in the pull request to work. This is shitty and lazy of me, which leads me to the question would someone like to take over this project? So that it can receive more attention that what i'm giving it.

mrdougal avatar Oct 24 '15 02:10 mrdougal

I'm happy to take it over, let me just look into what's involved before committing to it. I've never build a tmbundle before.

brianviveiros avatar Oct 26 '15 01:10 brianviveiros