puppet-ghostbuster 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 908643e9ab2393c19fe67d3b59c65dd9c83be53a
-  data.tar.gz: c8308c83ec8baea56fbcd6de574b4a0a614e2981
+  metadata.gz: 65c4533a993b62ad0194d46774be738437de8a38
+  data.tar.gz: d68124e057a0aee869af77cbf7923b8d3a47eaa3
 SHA512:
-  metadata.gz: fa0254d403fb3573c54bbe2c2dceafac7c306f2974685d9fbea83716a980270b35a14aa7403934f962bf364c75e74c115a6e5743d730daceb12d4feb4c4ce246
-  data.tar.gz: 7af1016cb3e9dc6169300fe854182da081d6f139fbfd1f0c4e9abffecb2684a95ed148598a810574b3513489fca4211cdae7dcf979b3b99f07cfe9b3c40a2319
+  metadata.gz: af14464b230b01dbde502d0f69f94d0be802ade503bf0067d660f557189419388534e2f936438a0249d480447ba2384a399a60c383530dfad2bcc2c9ff7319e7
+  data.tar.gz: 2884410e432f51d41c39bcb0f2e21cd164f788de919dd66c7057bc4842ac1bb71be20e90699140c0f78de96459d7958a1811077c653c46aed6956dbc8dbf434f

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Change Log
 
+## [0.4.0](https://rubygems.org/gems/puppet-ghostbuster/versions/0.4.0) (2016-04-28)
+[Full Changelog](https://github.com/camptocamp/puppet-ghostbuster/compare/0.3.0...0.4.0)
+
+**Implemented enhancements:**
+
+- Output to JSON[\#17](https://github.com/camptocamp/puppet-ghostbuster/pull/17) ([mcanevet](https://github.com/mcanevet))
+
 ## [0.3.0](https://rubygems.org/gems/puppet-ghostbuster/versions/0.3.0) (2016-04-26)
 [Full Changelog](https://github.com/camptocamp/puppet-ghostbuster/compare/0.2.1...0.3.0)
 

data/README.md

@@ -29,20 +29,45 @@ You can add preconnect command with ``-p`` and server url with ``-s``.
 Example output
 --------------
 ```
-$ bundle exec puppet-ghostbuster -s https://puppetdburl:8081
-Class Foo::Install not used
-Class Foo::Service not used
-Class Foo not used
-...
-Define Bar::Baz not used
-...
-Template ./modulename/templates/foo.erb not used
-Template ./modulename/templates/bar.erb not used
-Template ./modulename/templates/baz.erb not used
-...
-File ./foo/files/bar.txt not used
-File ./foo/files/baz.txt not used
-...
+$ puppet-ghostbuster -s https://puppetdburl:8081 | jsonpp
+[
+  {
+    "title": "[GhostBuster] Class Foo::Install seems unused",
+    "body": "./modules/foo/manifests/install.pp"
+  },
+  {
+    "title": "[GhostBuster] Class Foo::Service seems unused",
+    "body": "./modules/foo/manifests/service.pp"
+  },
+  {
+    "title": "[GhostBuster] Class Foo seems unused",
+    "body": "./modules/foo/manifests/init.pp"
+  },
+  {
+    "title": "[GhostBuster] Define Bar:Baz seems unused",
+    "body": "./modules/bar/manifests/baz.pp"
+  },
+  {
+    "title": "[GhostBuster] Template modulename/foo.erb seems unused",
+    "body": "./modules/modulename/templates/foo.erb"
+  },
+  {
+    "title": "[GhostBuster] Template modulename/bar.erb seems unused",
+    "body": "./modules/modulename/templates/bar.erb"
+  },
+  {
+    "title": "[GhostBuster] Template modulename/baz.erb seems unused",
+    "body": "./modules/modulename/templates/baz.erb"
+  },
+  {
+    "title": "[GhostBuster] File foo/bar.txt seems unused",
+    "body": "./modules/foo/files/bar.txt"
+  },
+  {
+    "title": "[GhostBuster] File foo/baz.txt seems unused",
+    "body": "./modules/foo/files/baz.txt"
+  }
+]
 ```
 
 How It Works

data/lib/puppet-ghostbuster/version.rb

@@ -1,3 +1,3 @@
 class PuppetGhostbuster
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end

data/lib/puppet-ghostbuster.rb

@@ -10,6 +10,7 @@ require 'puppet-ghostbuster/configuration'
 class PuppetGhostbuster
 
   attr_accessor :path
+  @@issues = []
 
   def exclude(filelist)
     ignorefile = "#{path}/.ghostbusterignore"
@@ -99,7 +100,7 @@ class PuppetGhostbuster
       if c = File.readlines(file).grep(/^class\s+([^\s\(\{]+)/){$1}[0]
         class_name = c.split('::').map(&:capitalize).join('::')
         count = self.class.used_classes.select { |klass| klass == class_name }.size
-        puts "Class #{class_name} not used" if count == 0
+        @@issues << { :title => "[GhostBuster] Class #{class_name} seems unused", :body => file } if count == 0
       end
     end
   end
@@ -114,7 +115,7 @@ class PuppetGhostbuster
       if d = File.readlines(file).grep(/^define\s+([^\s\(\{]+)/){$1}[0]
         define_name = d.split('::').map(&:capitalize).join('::')
         count = self.class.client.request('resources', [:'=', 'type', define_name]).data.size
-        puts "Define #{define_name} not used" if count == 0
+        @@issues << { :title => "[GhostBuster] Define #{define_name} seems unused", :body => file } if count == 0
       end
     end
   end
@@ -136,7 +137,7 @@ class PuppetGhostbuster
         end
         count += File.readlines(manifest).grep(/["']#{module_name}\/#{template_name}["']/).size
       end
-      puts "Template #{template} not used" if count == 0
+      @@issues << { :title => "[GhostBuster] Template #{module_name}/#{template_name} seems unused", :body => template } if count == 0
     end
   end
 
@@ -161,7 +162,7 @@ class PuppetGhostbuster
         rescue ArgumentError
         end
       end
-      puts "File #{file} not used" if count == 0
+      @@issues << { :title => "[GhostBuster] File #{module_name}/#{file_name} seems unused", :body => file } if count == 0
     end
   end
 
@@ -178,6 +179,7 @@ class PuppetGhostbuster
     find_unused_defines
     find_unused_templates
     find_unused_files
+    puts @@issues.to_json
   end
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puppet-ghostbuster
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Camptocamp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-26 00:00:00.000000000 Z
+date: 2016-04-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: coveralls