Merge pull request #46 from mtsmfm/rubocop

��Run Rubocop on CI

Author: mtsmfm

.gitignore

@@ -9,3 +9,4 @@
 /tmp/
 /vendor/bundle/
 /node_modules/
+/.rubocop-*

.rubocop.yml

@@ -0,0 +1,45 @@
+inherit_from:
+  - https://github.com/onk/onkcop/raw/v0.51.0.0/config/rubocop.yml
+
+AllCops:
+  TargetRubyVersion: 2.2
+  Exclude:
+    - 'bin/**/*'
+
+# TODO: Remove this config after droping Ruby 2.2 support
+Layout/IndentHeredoc:
+  Include:
+    - 'test/**/*'
+
+Lint/AssignmentInCondition:
+  Enabled: false
+
+Lint/HandleExceptions:
+  Enabled: false
+
+Naming/HeredocDelimiterNaming:
+  Enabled: false
+
+Style/BlockDelimiters:
+  Enabled: false
+
+Style/MultilineBlockChain:
+  Enabled: false
+
+Style/Semicolon:
+  Enabled: false
+
+Metrics/AbcSize:
+  Max: 50
+
+Metrics/BlockLength:
+  Max: 30
+
+Metrics/MethodLength:
+  Max: 40
+
+Metrics/ModuleLength:
+  Enabled: false
+
+Metrics/LineLength:
+  Max: 200

Gemfile

@@ -1,7 +1,7 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in language_server.gemspec
 gemspec
 
 # TODO: Use released gem after new release includes https://github.com/bbatsov/rubocop/pull/4987
-gem 'rubocop', git: 'https://github.com/bbatsov/rubocop'
+gem "rubocop", git: "https://github.com/bbatsov/rubocop"

Guardfile

@@ -5,5 +5,5 @@ guard :minitest, all_after_pass: true do
   watch(%r{^test/(.*)\/?test_(.*)\.rb$})
   watch(%r{^test/(.*)\/?(.*)_test\.rb$})
   watch(%r{^lib/(.*/)?([^/]+)\.rb$})     { |m| "test/#{m[1]}test_#{m[2]}.rb" }
-  watch(%r{^test/test_helper\.rb$})      { 'test' }
+  watch(%r{^test/test_helper\.rb$})      { "test" }
 end

Rakefile

@@ -4,7 +4,7 @@ require "rake/testtask"
 Rake::TestTask.new(:test) do |t|
   t.libs << "test"
   t.libs << "lib"
-  t.test_files = FileList['test/**/*_test.rb']
+  t.test_files = FileList["test/**/*_test.rb"]
 end
 
 task :default => :test

benchmark/rubocop

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 
-require 'bundler/setup'
-require 'benchmark/ips'
-require 'language_server'
+require "bundler/setup"
+require "benchmark/ips"
+require "language_server"
 
 error_code = <<-EOS
 require "foo
@@ -13,7 +13,7 @@ a = 1
 EOS
 valid_code = File.read(__FILE__)
 
-require 'pry-byebug'
+require "pry-byebug"
 
 LanguageServer::Linter::Rubocop.new(valid_code).call
 Benchmark.ips do |x|

benchmark/ruby_wc

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 
-require 'bundler/setup'
-require 'benchmark/ips'
-require 'language_server'
+require "bundler/setup"
+require "benchmark/ips"
+require "language_server"
 
 error_code = <<-EOS
 require "foo

bin/ci

@@ -0,0 +1,7 @@
+#!/usr/bin/env sh
+
+set -ex
+
+bin/setup
+bin/rubocop
+bin/m

bin/rubocop

@@ -0,0 +1,17 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+#
+# This file was generated by Bundler.
+#
+# The application 'rubocop' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile",
+  Pathname.new(__FILE__).realpath)
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("rubocop", "rubocop")

circle.yml

@@ -20,8 +20,7 @@ jobs:
             docker cp . project:/app
             docker start project
             docker-compose build $SERVICE_NAME
-      - run: docker-compose run $SERVICE_NAME bin/setup
-      - run: docker-compose run $SERVICE_NAME bin/m
+      - run: docker-compose run $SERVICE_NAME bin/ci
   test-2-3:
     <<: *test
     docker: