Rubocop formatting changes

Author: nat-henderson

assets/lib/filters/approval.rb

@@ -7,10 +7,10 @@ def initialize(pull_requests:, input: Input.instance)
 
     def pull_requests
       if @input.source.require_review_approval
-        @pull_requests.delete_if {|x| !x.review_approved? }
+        @pull_requests.delete_if { |x| !x.review_approved? }
       end
       if @input.source.authorship_restriction
-        @pull_requests.delete_if {|x| !x.author_associated? }
+        @pull_requests.delete_if { |x| !x.author_associated? }
       end
 
       @pull_requests

assets/lib/filters/mergeable.rb

@@ -1,13 +1,13 @@
 module Filters
-  class Mergeable 
+  class Mergeable
     def initialize(pull_requests:, input: Input.instance)
       @pull_requests = pull_requests
       @input = input
     end
 
     def pull_requests
       if @input.source.only_mergeable
-        @memoized ||= @pull_requests.delete_if {|x| !x.mergeable? }
+        @memoized ||= @pull_requests.delete_if { |x| !x.mergeable? }
       else
         @pull_requests
       end

assets/lib/pull_request.rb

@@ -21,14 +21,14 @@ def mergeable?
   end
 
   def review_approved?
-    Octokit.pull_request_reviews(base_repo, id).any? {|r| r['state'] == 'APPROVED'}
+    Octokit.pull_request_reviews(base_repo, id).any? { |r| r['state'] == 'APPROVED' }
   end
 
   def author_associated?
     # Checks whether the author is associated with the repo that the PR is against:
     # either the owner of that repo, someone invited to collaborate, or a member
     # of the organization who owns that repository.
-    %w(OWNER COLLABORATOR MEMBER).include? @pr['author_association']
+    %w[OWNER COLLABORATOR MEMBER].include? @pr['author_association']
   end
 
   def equals?(id:, sha:)

spec/commands/check_spec.rb

@@ -87,9 +87,9 @@ def stub_prs(uri, body)
   context 'when there is more than one open pull request' do
     before do
       stub_prs('https://api.github.com/repos/jtarchie/test/pulls?direction=asc&per_page=100&sort=updated&state=open', [
-                  { number: 1, head: { sha: 'abcdef', repo: { full_name: 'jtarchie/test' } }, base: { repo: { full_name: 'jtarchie/test' } } },
-                  { number: 2, head: { sha: 'zyxwvu', repo: { full_name: 'someotherowner/repo' } }, base: { repo: { full_name: 'jtarchie/test' } } }
-                ])
+                 { number: 1, head: { sha: 'abcdef', repo: { full_name: 'jtarchie/test' } }, base: { repo: { full_name: 'jtarchie/test' } } },
+                 { number: 2, head: { sha: 'zyxwvu', repo: { full_name: 'someotherowner/repo' } }, base: { repo: { full_name: 'jtarchie/test' } } }
+               ])
     end
 
     it 'returns all PRs oldest to newest last' do

spec/filters/approval_spec.rb

@@ -6,12 +6,12 @@
 describe Filters::Approval do
   let(:ignore_pr) do
     PullRequest.new(pr: { 'number' => 1, 'head' => { 'sha' => 'abc' }, 'author_association' => 'NONE',
-                          'base' => { 'repo' => {'full_name' => 'user/repo', 'permissions' => {'push' => true} } } })
+                          'base' => { 'repo' => { 'full_name' => 'user/repo', 'permissions' => { 'push' => true } } } })
   end
 
   let(:pr) do
     PullRequest.new(pr: { 'number' => 2, 'head' => { 'sha' => 'def' }, 'author_association' => 'OWNER',
-                          'base' => { 'repo' => {'full_name' => 'user/repo', 'permissions' => {'push' => true} } } })
+                          'base' => { 'repo' => { 'full_name' => 'user/repo', 'permissions' => { 'push' => true } } } })
   end
 
   let(:pull_requests) { [ignore_pr, pr] }
@@ -36,7 +36,7 @@ def stub_json(uri, body)
       expect(filter.pull_requests).to eq pull_requests
     end
   end
-  
+
   context 'when owner filtering is enabled' do
     it 'only returns PRs that are repo-owners' do
       payload = { 'source' => { 'repo' => 'user/repo', 'require_manual_approval' => false, 'require_review_approval' => false, 'authorship_restriction' => true } }
@@ -59,5 +59,4 @@ def stub_json(uri, body)
       expect(filter.pull_requests).to eq [pr]
     end
   end
-
 end

spec/filters/mergeable_spec.rb

@@ -5,11 +5,11 @@
 
 describe Filters::Mergeable do
   let(:ignore_pr) do
-    PullRequest.new(pr: { 'number' => 1, 'head' => { 'sha' => 'abc' }, 'mergeable' => false})
+    PullRequest.new(pr: { 'number' => 1, 'head' => { 'sha' => 'abc' }, 'mergeable' => false })
   end
 
   let(:pr) do
-    PullRequest.new(pr: { 'number' => 2, 'head' => { 'sha' => 'def' }, 'mergeable' => true})
+    PullRequest.new(pr: { 'number' => 2, 'head' => { 'sha' => 'def' }, 'mergeable' => true })
   end
 
   let(:pull_requests) { [ignore_pr, pr] }