additionally check Vagrantfile and gemspec files

Author: mschuchard

.circleci/config.yml

@@ -20,7 +20,7 @@ jobs:
         command: apt-get update && apt-get install -y cmake pkg-config
     - run:
         name: bundler install
-        command: bundle install --retry=3 && bundle binstubs puppet
+        command: bundle install --retry=3
     - save_cache:
         paths:
         - ~/.bundle

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### 2.3.1 (Next)
 - No input target paths now defaults to current working directory instead of error.
+- Additionally check `Vagrantfile` and `gemspec` files.
 
 ### 2.3.0
 - Minimum Ruby version increased to 2.7.

lib/puppet_check.rb

@@ -143,7 +143,7 @@ def execute_parsers(files, style, puppetlint_args, rubocop_args, public, private
     eyamls, files = files.partition { |file| File.extname(file) =~ /\.eya?ml$/ }
     DataParser.eyaml(eyamls, public, private) unless eyamls.empty?
     # check misc ruby
-    librarians, files = files.partition { |file| File.basename(file) =~ /(?:Puppet|Module|Rake|Gem)file$/ }
+    librarians, files = files.partition { |file| File.basename(file) =~ /(?:Puppet|Module|Rake|Gem|Vagrant)file|.gemspec$/ }
     RubyParser.librarian(librarians, style, rubocop_args) unless librarians.empty?
     # ignore everything else
     files.each { |file| self.class.files[:ignored].push(file.to_s) }

spec/puppet-check/cli_spec.rb

@@ -5,8 +5,8 @@
   context '.run' do
     it 'targets the current working directory if no paths were specified' do
       expect { PuppetCheck::CLI.run(%w[--fail-on-warnings]) }.not_to raise_exception
-      expect(PuppetCheck.files[:clean].length).to eql(28)
-      expect(PuppetCheck.files[:ignored].length).to eql(10)
+      expect(PuppetCheck.files[:clean].length).to eql(30)
+      expect(PuppetCheck.files[:ignored].length).to eql(7)
     end
   end