bibliothecary 4.0.4 → 5.0.0

data/lib/bibliothecary/parsers/packagist.rb

@@ -5,21 +5,15 @@ module Bibliothecary
     class Packagist
       include Bibliothecary::Analyser
 
-      def self.parse(filename, path)
-        if filename.match(/^composer\.json$/)
-          file_contents = File.open(path).read
-          json = JSON.parse(file_contents)
-          parse_manifest(json)
-        elsif filename.match(/^composer\.lock$/)
-          file_contents = File.open(path).read
-          json = JSON.parse(file_contents)
-          parse_lockfile(json)
-        else
-          []
-        end
+      def self.mapping
+        {
+          /^composer\.json$/ => :parse_manifest,
+          /^composer\.lock$/ => :parse_lockfile
+        }
       end
 
-      def self.parse_lockfile(manifest)
+      def self.parse_lockfile(file_contents)
+        manifest = JSON.parse file_contents
         manifest.fetch('packages',[]).map do |dependency|
           {
             name: dependency["name"],
@@ -29,7 +23,8 @@ module Bibliothecary
         end
       end
 
-      def self.parse_manifest(manifest)
+      def self.parse_manifest(file_contents)
+        manifest = JSON.parse file_contents
         map_dependencies(manifest, 'require', 'runtime') +
         map_dependencies(manifest, 'require-dev', 'development')
       end

data/lib/bibliothecary/parsers/pub.rb

@@ -5,26 +5,21 @@ module Bibliothecary
     class Pub
       include Bibliothecary::Analyser
 
-      def self.parse(filename, path)
-        if filename.match(/^pubspec\.yaml$/i)
-          file_contents = File.open(path).read
-          yaml = YAML.load file_contents
-          parse_yaml_manifest(yaml)
-        elsif filename.match(/^pubspec\.lock$/i)
-          file_contents = File.open(path).read
-          yaml = YAML.load file_contents
-          parse_yaml_lockfile(yaml)
-        else
-          []
-        end
+      def self.mapping
+        {
+          /^pubspec\.yaml$/i => :parse_yaml_manifest,
+          /^pubspec\.lock$/i => :parse_yaml_lockfile
+        }
       end
 
-      def self.parse_yaml_manifest(manifest)
+      def self.parse_yaml_manifest(file_contents)
+        manifest = YAML.load file_contents
         map_dependencies(manifest, 'dependencies', 'runtime') +
         map_dependencies(manifest, 'dev_dependencies', 'development')
       end
 
-      def self.parse_yaml_lockfile(manifest)
+      def self.parse_yaml_lockfile(file_contents)
+        manifest = YAML.load file_contents
         manifest.fetch('packages', []).map do |name, dep|
           {
             name: name,

data/lib/bibliothecary/parsers/pypi.rb

@@ -7,19 +7,18 @@ module Bibliothecary
       REQUIRE_REGEXP = /([a-zA-Z0-9]+[a-zA-Z0-9\-_\.]+)([><=\d\.,]+)?/
       REQUIREMENTS_REGEXP = /^#{REQUIRE_REGEXP}/
 
-      def self.parse(filename, path)
-        is_valid_requirements_file = is_requirements_file(filename)
-        if is_valid_requirements_file
-          file_contents = File.open(path).read
-          parse_requirements_txt(file_contents)
+      def self.parse_file(filename, contents)
+        if is_requirements_file(filename)
+          parse_requirements_txt(contents)
         elsif filename.match(/setup\.py$/)
-          file_contents = File.open(path).read
-          parse_setup_py(file_contents)
-        else
-          []
+          parse_setup_py(contents)
         end
       end
 
+      def self.match?(filename)
+        is_requirements_file(filename) || filename.match(/setup\.py$/)
+      end
+
       def self.parse_setup_py(manifest)
         match = manifest.match(INSTALL_REGEXP)
         return [] unless match
@@ -52,7 +51,6 @@ module Bibliothecary
       end
 
       def self.is_requirements_file(filename)
-        is_requirements_file = filename.match(/require.*\.(txt|pip)$/)
         if filename.match(/require.*\.(txt|pip)$/) and !filename.match(/^node_modules/)
           return true
         else

data/lib/bibliothecary/parsers/rubygems.rb

@@ -8,21 +8,12 @@ module Bibliothecary
       NAME_VERSION = '(?! )(.*?)(?: \(([^-]*)(?:-(.*))?\))?'.freeze
       NAME_VERSION_4 = /^ {4}#{NAME_VERSION}$/
 
-      def self.parse(filename, path)
-        if filename.match(/^Gemfile$|^gems\.rb$/)
-          file_contents = File.open(path).read
-          manifest = Gemnasium::Parser.send(:gemfile, file_contents)
-          parse_manifest(manifest)
-        elsif filename.match(/[A-Za-z0-9_-]+\.gemspec$/)
-          file_contents = File.open(path).read
-          manifest = Gemnasium::Parser.send(:gemspec, file_contents)
-          parse_manifest(manifest)
-        elsif filename.match(/^Gemfile\.lock$|^gems\.locked$/)
-          file_contents = File.open(path).read
-          parse_gemfile_lock(file_contents)
-        else
-          []
-        end
+      def self.mapping
+        {
+          /^Gemfile$|^gems\.rb$/ => :parse_gemfile,
+          /[A-Za-z0-9_-]+\.gemspec$/ => :parse_gemspec,
+          /^Gemfile\.lock$|^gems\.locked$/ => :parse_gemfile_lock
+        }
       end
 
       def self.parse_gemfile_lock(manifest)
@@ -39,15 +30,17 @@ module Bibliothecary
         end.compact
       end
 
-      def self.parse_manifest(manifest)
-        manifest.dependencies.inject([]) do |deps, dep|
-          deps.push({
-            name: dep.name,
-            requirement: dep.requirement.to_s,
-            type: dep.type
-          })
-        end.uniq
+      def self.parse_gemfile(file_contents)
+        manifest = Gemnasium::Parser.send(:gemfile, file_contents)
+        parse_ruby_manifest(manifest)
       end
+
+      def self.parse_gemspec(file_contents)
+        manifest = Gemnasium::Parser.send(:gemspec, file_contents)
+        parse_ruby_manifest(manifest)
+      end
+
+
     end
   end
 end

data/lib/bibliothecary/parsers/shard.rb

@@ -5,25 +5,20 @@ module Bibliothecary
     class Shard
       include Bibliothecary::Analyser
 
-      def self.parse(filename, path)
-        if filename.match(/^shard\.yml$/i)
-          file_contents = File.open(path).read
-          yaml = YAML.load file_contents
-          parse_yaml_manifest(yaml)
-        elsif filename.match(/^shard\.lock$/i)
-          file_contents = File.open(path).read
-          yaml = YAML.load file_contents
-          parse_yaml_lockfile(yaml)
-        else
-          []
-        end
+      def self.mapping
+        {
+          /^shard\.yml$/i => :parse_yaml_manifest,
+          /^shard\.lock$/i => :parse_yaml_lockfile
+        }
       end
 
-      def self.parse_yaml_lockfile(manifest)
+      def self.parse_yaml_lockfile(file_contents)
+        manifest = YAML.load file_contents
         map_dependencies(manifest, 'shards', 'runtime')
       end
 
-      def self.parse_yaml_manifest(manifest)
+      def self.parse_yaml_manifest(file_contents)
+        manifest = YAML.load file_contents
         map_dependencies(manifest, 'dependencies', 'runtime') +
         map_dependencies(manifest, 'development_dependencies', 'runtime')
       end

data/lib/bibliothecary/parsers/swift_pm.rb

@@ -3,13 +3,10 @@ module Bibliothecary
     class SwiftPM
       include Bibliothecary::Analyser
 
-      def self.parse(filename, path)
-        if filename.match(/^Package\.swift$/i)
-          file_contents = File.open(path).read
-          parse_package_swift(file_contents)
-        else
-          []
-        end
+      def self.mapping
+        {
+          /^Package\.swift$/i => :parse_package_swift
+        }
       end
 
       def self.parse_package_swift(manifest)

data/lib/bibliothecary/version.rb

@@ -1,3 +1,3 @@
 module Bibliothecary
-  VERSION = "4.0.4"
+  VERSION = "5.0.0"
 end

data/lib/sdl_parser.rb

@@ -7,10 +7,6 @@ class SdlParser
     @type = type || 'runtime'
   end
 
-  def to_json
-    Oj.dump(dependencies, mode: :compat) unless contents.nil?
-  end
-
   def dependencies
     parse.children('dependency').inject([]) do |deps, dep|
       deps.push({

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bibliothecary
 version: !ruby/object:Gem::Version
-  version: 4.0.4
+  version: 5.0.0
 platform: ruby
 authors:
 - Andrew Nesbitt
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-12-16 00:00:00.000000000 Z
+date: 2017-02-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: toml-rb
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.0'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.0'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -143,11 +143,15 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".codeclimate.yml"
+- ".github/CONTRIBUTING.md"
+- ".github/ISSUE_TEMPLATE.md"
+- ".github/PULL_REQUEST_TEMPLATE.md"
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - ".travis.yml"
 - CODE_OF_CONDUCT.md
-- CONTRIBUTING.md
 - Gemfile
 - LICENSE.txt
 - README.md

data/CONTRIBUTING.md

@@ -1,10 +0,0 @@
-# Contributing
-
- * Fork the project.
- * Make your feature addition or bug fix.
- * Add tests for it. This is important so I don't break it in a future version unintentionally.
- * Send a pull request. Bonus points for topic branches.
-
-## Code of Conduct
-
-Please note that this project is released with a [Contributor Code of Conduct](CODE_OF_CONDUCT.md). By participating in this project you agree to abide by its terms.