coconductor 0.9.1 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9fd3d9f82c0aec28b73f9f692226a6ea3a962ff13deb2315e520363afc73c3ae
-  data.tar.gz: 0023d2b81bc25e49604bf25828e84a30f92b5de5bb4f1c3ba90dafb56254e680
+  metadata.gz: f2f164d3139954a092f0e47c1a7d3ed25988927d1dc6c789c0c87906a370500c
+  data.tar.gz: ee301805000f620f4def3f84aea18302154aefb2bad3a43d162adf159b4832ac
 SHA512:
-  metadata.gz: 03324bbb7084bf316394ae361cba6d7ca844c825b87525f7f78e06c4c82642bd911defd2f0f06c7f979c8731dc4d29c392a9e318626007649172e0a50ff643a8
-  data.tar.gz: c63dc0a0abfea44a913d11435370ed7044c70985136ccc3ee8c113fd03f0e4b1b275a412a0592d4424dc806677aea1e1984b54cdc3d3d292209b950af1a6623a
+  metadata.gz: 94d375ee6d990acd446063a34e93d28cbd93843e76d30b571f435af1eb8e2925bf9c0e7f6fe554499200eed3fba2425b505842f343afc5c99bea9983a071afb7
+  data.tar.gz: caebc5d1f2743c2c08710a2c5d88bf7798c1307ee51144ba6b08add09adf0f806c9fbabaf15d1e6c1c68630926808b02e3045f1d082efe70b8a89a87276d2ec1

data/.github/funding.yml

@@ -0,0 +1 @@
+patreon: benbalter

data/.rubocop.yml

@@ -1,3 +1,12 @@
+inherit_from: .rubocop_todo.yml
+
+require:
+  - rubocop-performance
+  - rubocop-rspec
+
+AllCops:
+  NewCops: enable
+
 Style/Documentation:
   Enabled: false
 
@@ -6,7 +15,7 @@ Metrics/BlockLength:
     - spec/**/*
     - coconductor.gemspec
 
-Metrics/LineLength:
+Layout/LineLength:
   Exclude:
     - lib/coconductor/project_files.rb
     - lib/coconductor/commands/*

data/.rubocop_todo.yml

@@ -0,0 +1,132 @@
+# This configuration was generated by
+# `rubocop --auto-gen-config`
+# on 2021-03-02 20:51:18 UTC using RuboCop version 1.11.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+# Configuration parameters: Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'coconductor.gemspec'
+
+# Offense count: 1
+Lint/NoReturnInBeginEndBlocks:
+  Exclude:
+    - 'lib/coconductor/field.rb'
+
+# Offense count: 1
+# Configuration parameters: CountComments, CountAsOne, ExcludedMethods, IgnoredMethods.
+Metrics/MethodLength:
+  Max: 11
+
+# Offense count: 7
+# Configuration parameters: EnforcedStyle, CheckMethodNames, CheckSymbols, AllowedIdentifiers.
+# SupportedStyles: snake_case, normalcase, non_integer
+# AllowedIdentifiers: capture3, iso8601, rfc1123_date, rfc822, rfc2822, rfc3339
+Naming/VariableNumber:
+  Exclude:
+    - 'spec/coconductor/matchers/dice_spec.rb'
+    - 'spec/coconductor/matchers/exact_spec.rb'
+    - 'spec/coconductor/project_files/code_of_conduct_file_spec.rb'
+    - 'spec/coconductor/project_files/project_file_spec.rb'
+    - 'spec/coconductor/project_spec.rb'
+    - 'spec/coconductor_spec.rb'
+    - 'spec/integration_spec.rb'
+
+# Offense count: 4
+# Configuration parameters: MinSize.
+Performance/CollectionLiteralInLoop:
+  Exclude:
+    - 'spec/coconductor/code_of_conduct_spec.rb'
+    - 'spec/coconductor/field_spec.rb'
+    - 'spec/coconductor/matchers/field_aware_spec.rb'
+    - 'spec/vendored_coc_spec.rb'
+
+# Offense count: 21
+# Configuration parameters: Prefixes.
+# Prefixes: when, with, without
+RSpec/ContextWording:
+  Exclude:
+    - 'spec/coconductor/code_of_conduct_spec.rb'
+    - 'spec/coconductor/field_spec.rb'
+    - 'spec/coconductor/matchers/dice_spec.rb'
+    - 'spec/coconductor/matchers/field_aware_spec.rb'
+    - 'spec/coconductor/project_files/code_of_conduct_file_spec.rb'
+    - 'spec/coconductor/project_files/project_file_spec.rb'
+    - 'spec/coconductor/project_spec.rb'
+    - 'spec/coconductor_spec.rb'
+    - 'spec/integration_spec.rb'
+
+# Offense count: 3
+# Configuration parameters: IgnoredMetadata.
+RSpec/DescribeClass:
+  Exclude:
+    - 'spec/coconductor/bin_spec.rb'
+    - 'spec/integration_spec.rb'
+    - 'spec/vendored_coc_spec.rb'
+
+# Offense count: 3
+# Configuration parameters: Max.
+RSpec/ExampleLength:
+  Exclude:
+    - 'spec/coconductor/code_of_conduct_spec.rb'
+
+# Offense count: 19
+RSpec/MultipleExpectations:
+  Max: 4
+
+# Offense count: 10
+# Configuration parameters: AllowSubject.
+RSpec/MultipleMemoizedHelpers:
+  Max: 8
+
+# Offense count: 71
+# Configuration parameters: IgnoreSharedExamples.
+RSpec/NamedSubject:
+  Exclude:
+    - 'spec/coconductor/code_of_conduct_spec.rb'
+    - 'spec/coconductor/field_spec.rb'
+    - 'spec/coconductor/matchers/dice_spec.rb'
+    - 'spec/coconductor/matchers/exact_spec.rb'
+    - 'spec/coconductor/matchers/matcher_spec.rb'
+    - 'spec/coconductor/project_files/project_file_spec.rb'
+    - 'spec/coconductor/project_spec.rb'
+    - 'spec/coconductor_spec.rb'
+    - 'spec/integration_spec.rb'
+
+# Offense count: 7
+RSpec/NestedGroups:
+  Max: 4
+
+# Offense count: 2
+RSpec/RepeatedDescription:
+  Exclude:
+    - 'spec/coconductor/code_of_conduct_spec.rb'
+
+# Offense count: 9
+RSpec/ScatteredSetup:
+  Exclude:
+    - 'spec/coconductor/project_files/project_file_spec.rb'
+    - 'spec/coconductor/project_spec.rb'
+    - 'spec/integration_spec.rb'
+
+# Offense count: 5
+RSpec/SubjectStub:
+  Exclude:
+    - 'spec/coconductor/project_files/project_file_spec.rb'
+
+# Offense count: 1
+Security/Open:
+  Exclude:
+    - 'lib/coconductor/vendorer.rb'
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: Methods.
+Style/RedundantArgument:
+  Exclude:
+    - 'spec/coconductor/project_files/project_file_spec.rb'

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 

data/bin/coconductor

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require_relative '../lib/coconductor'
 require 'thor'
@@ -38,6 +39,6 @@ class CoconductorCLI < Thor
 end
 
 commands_dir = File.expand_path '../lib/coconductor/commands/', __dir__
-Dir["#{commands_dir}/*.rb"].each { |c| require(c) }
+Dir["#{commands_dir}/*.rb"].sort.each { |c| require(c) }
 
 CoconductorCLI.start(ARGV)

data/coconductor.gemspec

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'coconductor/version'
@@ -24,14 +26,16 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'licensee', '~> 9.9', '>= 9.9.4'
-  spec.add_dependency 'thor', '~> 0.18'
+  spec.add_dependency 'thor', '>= 0.18', '< 2.0'
   spec.add_dependency 'toml', '~> 0.2'
 
-  spec.add_development_dependency 'bundler', '~> 1.16'
+  spec.add_development_dependency 'gem-release', '~> 2.0'
   spec.add_development_dependency 'pry', '~> 0.10'
   spec.add_development_dependency 'reverse_markdown', '~> 1.1'
   spec.add_development_dependency 'rspec', '~> 3.0'
-  spec.add_development_dependency 'rubocop', '~> 0.50'
+  spec.add_development_dependency 'rubocop', '~> 1.0'
+  spec.add_development_dependency 'rubocop-performance', '~> 1.5'
+  spec.add_development_dependency 'rubocop-rspec', '~> 2.0'
   spec.add_development_dependency 'twitter-text', '< 2.0'
   spec.add_development_dependency 'webmock', '~> 3.1'
   spec.add_development_dependency 'wikicloth', '~> 0.8'

data/docs/SECURITY.md

@@ -0,0 +1,3 @@
+# Security Policy
+
+To report a security vulnerability, please email [ben@balter.com](mailto:ben@balter.com).

data/lib/coconductor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative './coconductor/version'
 require 'licensee'
 
@@ -9,7 +11,7 @@ module Coconductor
   autoload :ProjectFiles,  'coconductor/project_files'
   autoload :Vendorer,      'coconductor/vendorer'
 
-  CONFIDENCE_THRESHOLD = 90
+  CONFIDENCE_THRESHOLD = 85
 
   class << self
     attr_writer :confidence_threshold
@@ -23,13 +25,13 @@ module Coconductor
     end
 
     def project(path, **args)
-      if path =~ %r{\Ahttps://github.com}
-        Coconductor::Projects::GitHubProject.new(path, args)
+      if %r{\Ahttps://github.com}.match?(path)
+        Coconductor::Projects::GitHubProject.new(path, **args)
       else
-        Coconductor::Projects::GitProject.new(path, args)
+        Coconductor::Projects::GitProject.new(path, **args)
       end
     rescue Coconductor::Projects::GitProject::InvalidRepository
-      Coconductor::Projects::FSProject.new(path, args)
+      Coconductor::Projects::FSProject.new(path, **args)
     end
 
     def confidence_threshold

data/lib/coconductor/code_of_conduct.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 require 'toml'
 
 module Coconductor
   class InvalidCodeOfConduct < ArgumentError; end
+
   class CodeOfConduct
     class << self
       def all
@@ -37,7 +40,7 @@ module Coconductor
           [
             matches['family'], 'version', matches['version'], matches['lang']
           ].compact.join('/')
-        end.compact
+        end.compact.sort
       end
 
       def latest_in_family(family, language: DEFAULT_LANGUAGE)
@@ -50,7 +53,7 @@ module Coconductor
       def families
         @families ||= begin
           families = Dir["#{vendor_dir}/*"].map { |dir| File.basename(dir) }
-          families - ['bundle']
+          (families - ['bundle']).sort
         end
       end
 
@@ -70,17 +73,18 @@ module Coconductor
       /version
       /(?<version>(?<major>\d)/(?<minor>\d)(/(?<patch>\d))?|(longer|shorter))
       /#{Coconductor::ProjectFiles::CodeOfConductFile::FILENAME_REGEX}
-    }ix
-    DEFAULT_LANGUAGE = 'en'.freeze
+    }ix.freeze
+    DEFAULT_LANGUAGE = 'en'
 
     attr_reader :key
     attr_writer :content
+
     include Licensee::ContentHelper
 
     # Define dynamic predicate helpers to determine if a code of conduct is a
     # member of a given family, e.g., code_of_conduct.contributor_covenant?
     CodeOfConduct.families.each do |f|
-      define_method(f.tr('-', '_') + '?') { family == f }
+      define_method("#{f.tr('-', '_')}?") { family == f }
     end
 
     def initialize(key)
@@ -96,7 +100,7 @@ module Coconductor
         parts = key.split('/')
         if pseudo?
           nil
-        elsif parts.last =~ /^[a-z-]{2,5}$/
+        elsif /^[a-z-]{2,5}$/.match?(parts.last)
           parts.last
         else
           DEFAULT_LANGUAGE
@@ -130,7 +134,7 @@ module Coconductor
     alias body content
 
     def inspect
-      "#<Licensee::CodeOfConduct key=#{key}>"
+      "#<Coconductor::CodeOfConduct key=#{key}>"
     end
 
     def family
@@ -177,16 +181,18 @@ module Coconductor
     private
 
     def filename
-      filename = if contributor_covenant?
+      filename = if contributor_covenant? && key.include?('/version/1/')
                    'code-of-conduct'
+                 elsif contributor_covenant? && key.include?('/version/2/')
+                   'code_of_conduct'
                  elsif citizen_code_of_conduct?
                    'citizen_code_of_conduct'
                  else
                    'CODE_OF_CONDUCT'
                  end
 
-      filename << '.' + language unless default_language?
-      filename << '.md'
+      filename += ".#{language}" unless default_language?
+      "#{filename}.md"
     end
 
     def filepath

data/lib/coconductor/commands/detect.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CoconductorCLI < Thor
   desc 'detect [PATH]', 'Detect the code of conduct of the given project', default: Dir.pwd
   option :confidence, type: :numeric, default: Coconductor.confidence_threshold, desc: 'Confidence threshold'
@@ -57,7 +59,7 @@ class CoconductorCLI < Thor
     when :confidence
       Licensee::ContentHelper.format_percent(value)
     when :method
-      value.to_s.tr('_', ' ').capitalize + ':'
+      "#{value.to_s.tr('_', ' ').capitalize}:"
     else
       value
     end

data/lib/coconductor/commands/diff.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'tmpdir'
 
 class CoconductorCLI < Thor
@@ -39,20 +41,18 @@ class CoconductorCLI < Thor
   private
 
   def code_of_conduct_to_diff
-    if options[:code_of_conduct_to_diff]
-      return options[:code_of_conduct_to_diff]
-    end
+    return options[:code_of_conduct_to_diff] if options[:code_of_conduct_to_diff]
 
     return project.code_of_conduct_file if remote?
 
     @code_of_conduct_to_diff ||= begin
-      if STDIN.tty?
+      if $stdin.tty?
         error 'You must pipe the file contents to the command via STDIN'
         exit 1
       end
 
       filename = 'CODE_OF_CONDUCT.txt'
-      Coconductor::ProjectFiles::CodeOfConductFile.new(STDIN.read, filename)
+      Coconductor::ProjectFiles::CodeOfConductFile.new($stdin.read, filename)
     end
   end
 

data/lib/coconductor/commands/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CoconductorCLI < Thor
   desc 'version', 'Return the Coconductor version'
   def version

data/lib/coconductor/field.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module Coconductor
   # Represents a fillable field in a code of conduct
   class Field
-    REGEX = /\[(?<name>[A-Z_ ]{2,})[\s:-]*(?<description>.*?)\]/
+    REGEX = /\[(?<name>[A-Z_ ]{2,})[\s:-]*(?<description>.*?)\]/.freeze
 
     # the matchable raw text within the code of conduct including brackets
     attr_reader :raw_text
@@ -18,7 +20,7 @@ module Coconductor
       # Returns an array of Fields for the given code of conduct
       def from_code_of_conduct(code_of_conduct)
         matches = []
-        return [] unless code_of_conduct && code_of_conduct.content
+        return [] unless code_of_conduct&.content
 
         code_of_conduct.content.scan(REGEX) do |_m|
           matches << Regexp.last_match
@@ -40,7 +42,7 @@ module Coconductor
     def initialize(raw_text, name: nil, description: nil)
       @raw_text = raw_text
       @name = name
-      @description = description if description && description != ''
+      @description = description.capitalize if description && description != ''
     end
 
     # The unformatted field name as found in the code of conduct text
@@ -60,9 +62,7 @@ module Coconductor
 
     def description
       @description ||= begin
-        if parts && parts[:description] && parts[:description] != ''
-          return parts[:description].capitalize
-        end
+        return parts[:description].capitalize if parts && parts[:description] && parts[:description] != ''
 
         DESCRIPTIONS[key]
       end