rubocop and buildifier style fixes

Author: graham jenson

.travis.yml

@@ -38,7 +38,9 @@ install:
 script:
 - bazel $BAZEL_OPTS test $BAZEL_BUILD_OPTS --show_progress_rate_limit 0 --test_output=streamed -- //...
 - (cd examples && bazel $BAZEL_OPTS test $BAZEL_BUILD_OPTS --show_progress_rate_limit 0 --test_output=streamed -- //...)
-- bazel run :buildifier
+# If this fails run `bazel run :buildifier`
+- bazel run :buildifier-check
+# If this fails run `bundle exec rubocop -a`
 - bundle exec rubocop
 
 env:

Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
-gem 'rubocop', '~> 0.76.0', require: false
+gem 'rubocop', '~> 0.76.0', require: false

examples/Gemfile

@@ -2,6 +2,6 @@
 
 source "https://rubygems.org"
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 gem 'rspec', '~> 3.7.0'

examples/bin/show_version.rb

@@ -1,6 +1,7 @@
 #!/usr/bin/ruby
+# frozen_string_literal: true
 
 require 'foo'
 
-bar = Foo::Bar.new
+Foo::Bar.new
 puts Foo::VERSION

examples/lib/foo.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Foo
   require 'foo/bar'
   require 'foo/version'

examples/lib/foo/bar.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'openssl'
 
 module Foo

examples/lib/foo/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Foo
   VERSION = "0.0.1"
 end

examples/spec/foo/version_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../spec_helper'
 require 'foo/version'
 

examples/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec.configure do |config|
   config.expect_with :rspec do |expectations|
     expectations.include_chain_clauses_in_custom_matcher_descriptions = true
@@ -11,7 +13,7 @@
 
   config.warnings = true
   config.filter_run_when_matching :focus
-  #config.disable_monkey_patching!
+  # config.disable_monkey_patching!
   config.order = :random
   Kernel.srand config.seed
 end

ruby/def.bzl

@@ -2,21 +2,20 @@ load(
     "@bazelruby_ruby_rules//ruby/private:library.bzl",
     _library = "ruby_library",
 )
-
 load(
     "@bazelruby_ruby_rules//ruby/private:binary.bzl",
     _binary = "ruby_binary",
     _test = "ruby_test",
 )
 
 def ruby_library(**attrs):
-  print("//ruby:def.bzl was deprecated and will be removed soon. Use //ruby:defs.bzl instead")
-  _library(**attrs)
+    print("//ruby:def.bzl was deprecated and will be removed soon. Use //ruby:defs.bzl instead")
+    _library(**attrs)
 
 def ruby_binary(**attrs):
-  print("//ruby:def.bzl was deprecated and will be removed soon. Use //ruby:defs.bzl instead")
-  _binary(**attrs)
+    print("//ruby:def.bzl was deprecated and will be removed soon. Use //ruby:defs.bzl instead")
+    _binary(**attrs)
 
 def ruby_test(**attrs):
-  print("//ruby:def.bzl was deprecated and will be removed soon. Use //ruby:defs.bzl instead")
-  _test(**attrs)
+    print("//ruby:def.bzl was deprecated and will be removed soon. Use //ruby:defs.bzl instead")
+    _test(**attrs)