reckon 0.9.3 → 0.9.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bc9cf03c584638f19ffd111497105288a9002a71ec4dd649655a7fa1b8c6566c
-  data.tar.gz: 75453fe81b3f131a72dae8ab1f592cf0bcfb61fb380c5486cb984ffb504fa64b
+  metadata.gz: 489081763b4a46c0bad41cb3ff3435c505a413e3e2cdf39984ff51b6488bbcc2
+  data.tar.gz: c4f6cfc6bec319fd8366c5effabb6c23b086bd0b90a57f9f9f64742805a03fc1
 SHA512:
-  metadata.gz: 9cc31d230176f3d70609d7560ce5b948295d3eef33402ea12aa91749ecb48cf4c36e9885af4ea07e451c8641ae1acacd5421b093621cbde65f88da116992be30
-  data.tar.gz: d4e743fa0f3104bc4fe14774ea5e7ce8b7862c0865b725f6e7fac5f0a68607c7baa45c95a99d755c46a03dd90389d69e6addfd06c990103e4bc12c9959e6bf65
+  metadata.gz: b995bbf4939b6901fa769188a4f329a549d0ea20048a968c331fdce40c7e79174d65454d1805f5ba9a66717c7269afbdeef931af8e961a52e1327a7edbbc80e1
+  data.tar.gz: e5107209cf8e6da13c6484d064ec7e6e911502128d891d03590b04354ae5fa24670044e77aab2b90733882391cadb3a8adbd1a8645916b7dcb39d8811dba5070

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## [v0.9.5](https://github.com/cantino/reckon/tree/v0.9.5) (2024-01-08)
+
+[Full Changelog](https://github.com/cantino/reckon/compare/v0.9.4...v0.9.5)
+
+**Closed issues:**
+
+- Ruby Readline error [\#127](https://github.com/cantino/reckon/issues/127)
+- Date formats [\#126](https://github.com/cantino/reckon/issues/126)
+
+**Merged pull requests:**
+
+- Pin highline to 2.x branch. Fixes \#127 [\#128](https://github.com/cantino/reckon/pull/128) ([benprew](https://github.com/benprew))
+
+## [v0.9.4](https://github.com/cantino/reckon/tree/v0.9.4) (2023-11-24)
+
+[Full Changelog](https://github.com/cantino/reckon/compare/v0.9.3...v0.9.4)
+
 ## [v0.9.3](https://github.com/cantino/reckon/tree/v0.9.3) (2023-08-25)
 
 [Full Changelog](https://github.com/cantino/reckon/compare/v0.9.2...v0.9.3)

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    reckon (0.9.3)
+    reckon (0.9.5)
       chronic (>= 0.3.0)
-      highline (>= 1.5.2)
+      highline (~> 2.0)
       matrix (>= 0.4.2)
       rchardet (>= 1.8.0)
 

data/lib/reckon/date_column.rb

@@ -81,12 +81,15 @@ module Reckon
       date_score += 30 if entry =~ /^\d+[:\/\.-]\d+[:\/\.-]\d+([ :]\d+[:\/\.]\d+)?$/
       date_score += 10 if entry =~ /^\d+\[\d+:GMT\]$/i
 
+      # ruby 2.6.0 doesn't have Date::Error, but Date::Error is a subclass of
+      # ArgumentError
+      #
+      # Sometimes DateTime.parse can throw a RangeError
+      # See https://github.com/cantino/reckon/issues/126
       begin
         DateTime.parse(entry)
         date_score += 20
-      # ruby 2.6.0 doesn't have Date::Error, but Date::Error is a subclass of
-      # ArgumentError
-      rescue ArgumentError
+      rescue StandardError
         # we don't need do anything here since the column didn't parse as a date
         nil
       end

data/lib/reckon/version.rb

@@ -1,3 +1,3 @@
 module Reckon
-  VERSION = "0.9.3"
+  VERSION = "0.9.5"
 end

data/reckon.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency "pry", ">= 0.12.2"
   s.add_development_dependency "rantly", "= 1.2.0"
   s.add_runtime_dependency "chronic", ">= 0.3.0"
-  s.add_runtime_dependency "highline", ">= 1.5.2"
+  s.add_runtime_dependency "highline", "~> 2.0"  # 3.0 replaces readline with reline and breaks reckon
   s.add_runtime_dependency "rchardet", ">= 1.8.0"
   s.add_runtime_dependency "matrix", ">= 0.4.2"
 end

data/spec/reckon/date_column_spec.rb

@@ -1,59 +1,72 @@
 #!/usr/bin/env ruby
-# encoding: utf-8
+# frozen_string_literal: true
 
 require "spec_helper"
-require 'rubygems'
-require 'reckon'
-
-describe Reckon::DateColumn do
-  describe "initialize" do
-    it "should detect us and world time" do
-      Reckon::DateColumn.new( ["01/02/2013", "01/14/2013"] ).endian_precedence.should == [:middle]
-      Reckon::DateColumn.new( ["01/02/2013", "14/01/2013"] ).endian_precedence.should == [:little]
-    end
-    it "should set endian_precedence to default when date format cannot be misinterpreted" do
-      Reckon::DateColumn.new( ["2013/01/02"] ).endian_precedence.should == [:middle,:little]
-    end
-    it "should raise an error when in doubt" do
-      expect{ Reckon::DateColumn.new( ["01/02/2013", "01/03/2013"] )}.to raise_error( StandardError )
-    end
-  end
-  describe "for" do
-    it "should detect the date" do
-      expect(Reckon::DateColumn.new(%w[13/12/2013]).for(0))
-        .to eq(Date.new(2013, 12, 13))
-      expect(Reckon::DateColumn.new(%w[01/14/2013]).for(0))
-        .to eq(Date.new(2013, 1, 14))
-      expect(Reckon::DateColumn.new(%w[13/12/2013 21/11/2013]).for(1))
-        .to eq(Date.new(2013, 11, 21))
-      expect(Reckon::DateColumn.new( ["2013-11-21"] ).for( 0 ))
-        .to eq(Date.new(2013, 11, 21))
+require "rubygems"
+require "reckon"
 
+# datecolumn specs
+module Reckon
+  describe DateColumn do
+    describe "#initialize" do
+      it "should detect us and world time" do
+        expect(DateColumn.new(%w[01/02/2013 01/14/2013]).endian_precedence)
+          .to eq [:middle]
+        expect(DateColumn.new(%w[01/02/2013 14/01/2013]).endian_precedence)
+          .to eq [:little]
+      end
+      it "should set endian_precedence to default when date format cannot be misinterpreted" do
+        expect(DateColumn.new(["2013/01/02"]).endian_precedence)
+          .to eq %i[middle little]
+      end
+      it "should raise an error when in doubt" do
+        expect { DateColumn.new(["01/02/2013", "01/03/2013"]) }
+          .to raise_error(StandardError)
+      end
     end
 
-    it "should correctly use endian_precedence" do
-      expect(Reckon::DateColumn.new(%w[01/02/2013 01/14/2013]).for(0))
-        .to eq(Date.new(2013, 1, 2))
-      expect(Reckon::DateColumn.new(%w[01/02/2013 14/01/2013]).for(0))
-        .to eq(Date.new(2013, 2, 1))
-    end
-  end
+    describe "#for" do
+      it "should detect the date" do
+        expect(DateColumn.new(%w[13/12/2013]).for(0)).to eq(Date.new(2013, 12, 13))
+        expect(DateColumn.new(%w[01/14/2013]).for(0)).to eq(Date.new(2013, 1, 14))
+        expect(DateColumn.new(%w[13/12/2013 21/11/2013]).for(1))
+          .to eq(Date.new(2013, 11, 21))
+        expect(DateColumn.new(["2013-11-21"]).for(0)).to eq(Date.new(2013, 11, 21))
+      end
 
-  describe "#pretty_for" do
-    it 'should use ledger_date_format' do
-      expect(Reckon::DateColumn.new(%w[13/02/2013], {ledger_date_format: '%d/%m/%Y'}).pretty_for(0))
-               .to eq('13/02/2013')
+      it "should correctly use endian_precedence" do
+        expect(DateColumn.new(%w[01/02/2013 01/14/2013]).for(0))
+          .to eq(Date.new(2013, 1, 2))
+        expect(DateColumn.new(%w[01/02/2013 14/01/2013]).for(0))
+          .to eq(Date.new(2013, 2, 1))
+      end
     end
 
-    it 'should default to is' do
-      expect(Reckon::DateColumn.new(%w[13/12/2013]).pretty_for(0))
-        .to eq('2013-12-13')
+    describe "#pretty_for" do
+      it "should use ledger_date_format" do
+        expect(
+          DateColumn.new(["13/02/2013"],
+                         { ledger_date_format: "%d/%m/%Y" }).pretty_for(0)
+        )
+          .to eq("13/02/2013")
+      end
+
+      it "should default to is" do
+        expect(DateColumn.new(["13/12/2013"]).pretty_for(0))
+          .to eq("2013-12-13")
+      end
     end
-  end
 
-  describe "#likelihood" do
-    it "should prefer numbers that looks like dates" do
-      expect(Reckon::DateColumn.likelihood("123456789")).to be < Reckon::DateColumn.likelihood("20160102")
+    describe "#likelihood" do
+      it "should prefer numbers that looks like dates" do
+        expect(DateColumn.likelihood("123456789"))
+          .to be < DateColumn.likelihood("20160102")
+      end
+
+      # See https://github.com/cantino/reckon/issues/126
+      it "Issue #126 - it shouldn't fail on invalid dates" do
+        expect(DateColumn.likelihood("303909302970-07-2023")).to be > 0
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reckon
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.9.5
 platform: ruby
 authors:
 - Andrew Cantino
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-25 00:00:00.000000000 Z
+date: 2024-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -72,16 +72,16 @@ dependencies:
   name: highline
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.5.2
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.5.2
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: rchardet
   requirement: !ruby/object:Gem::Requirement