Merge branch 'jney-master' into new-api

.travis.yml

@@ -28,7 +28,8 @@ before_install:
   - brew tap homebrew/dupes
   - brew tap homebrew/versions
   # Ruby language support
-  - gem install ruby-beautify --verbose
+  # - gem install ruby-beautify --verbose
+  - gem install rubocop
   # Python language support
   - brew install python
   - pip install --upgrade autopep8

examples/simple-jsbeautifyrc/ruby/expected/test.rb

@@ -1,25 +1,24 @@
 #!/usr/bin/env ruby
 
-conn_hash = { :hosts => [
+conn_hash = { hosts: [
 
-                {:login => login, :passcode => passcode, :host => host, :port => port},
-              ],
-              :reliable => false, # Override default
-              :connect_headers => conn_hdrs,
-
-              }
-
-
-hash = { :hosts => [
-
-           {:login => user, :passcode => password, :host => 'noonehome', :port => 2525},
-
-           {:login => user, :passcode => password, :host => host, :port => port},
-
-         ],
-
-         :logger => mylog,	# This enables callback logging!
-
-         :max_reconnect_attempts => 5,
+  { login: login, passcode: passcode, host: host, port: port }
+],
+              reliable: false, # Override default
+              connect_headers: conn_hdrs
+
+}
+
+hash = { hosts: [
+
+  { login: user, passcode: password, host: 'noonehome', port: 2525 },
+
+  { login: user, passcode: password, host: host, port: port }
+
+],
+
+         logger: mylog,	# This enables callback logging!
+
+         max_reconnect_attempts: 5
 
         }

src/beautifiers/beautifier.coffee

@@ -40,10 +40,9 @@ module.exports = class Beautifier
 
     ###
     Show deprecation warning to user.
-
-    TODO: implement this
     ###
     deprecate: (warning) ->
+        atom.notifications?.addWarning(warning)
 
     ###
     Create temporary file
@@ -158,14 +157,15 @@ module.exports = class Beautifier
                         cmd.stderr.on('data', (data) -> stderr += data )
                         # when the spawn child process exits, check if there were any errors and close the writeable stream
                         cmd.on('exit', (code) ->
-                            # console.log('spawn done', code, stderr, stdout)
+                            console.log('spawn done', code, stderr, stdout)
                             # If return code is not 0 then error occured
-                            if code isnt 0
-                                reject(stderr)
-                            else
+                            # if code isnt 0
+                            #     reject(stderr)
+                            # else
                             resolve(stdout)
                         )
                         cmd.on('error', (err) ->
+                            console.log('error', err)
                             reject(err)
                         )
                     )

src/beautifiers/index.coffee

@@ -39,6 +39,7 @@ module.exports = class Beautifiers
         'perltidy'
         'php-cs-fixer'
         'prettydiff'
+        'rubocop'
         'ruby-beautify'
         'sqlformat'
         'tidy-markdown'

src/beautifiers/rubocop.coffee

@@ -0,0 +1,23 @@
+###
+Requires https://github.com/bbatsov/rubocop
+###
+
+"use strict"
+Beautifier = require('./beautifier')
+
+module.exports = class Rubocop extends Beautifier
+    name: "Rubocop"
+
+    options: {
+        Ruby: true
+    }
+
+    beautify: (text, language, options) ->
+        @run("rubocop", [
+                "--auto-correct"
+                tempFile = @tempFile("temp", text)
+            ])
+            .then(=>
+                console.log('rubocop', arguments, tempFile)
+                @readFile(tempFile)
+            )

src/beautifiers/ruby-beautify.coffee

@@ -13,6 +13,7 @@ module.exports = class RubyBeautify extends Beautifier
     }
 
     beautify: (text, language, options) ->
+        @deprecate("Ruby-Beautify has been deprecated in favour of Rubocop beautifier.")
         @run("rbeautify", [
                 @tempFile("input", text)
             ])