fix findall method ambiguity in Julia 1.7 (#23)

* fix findall method ambiguity in Julia 1.7

* patch bump

* update CI and drop julia 1.0 support

* fix whitespace error

.github/workflows/CI.yml

@@ -12,8 +12,8 @@ jobs:
       fail-fast: false
       matrix:
         version:
-          - '1.0'
-          - '1'
+          - '1.6' # LTS
+          - '1'   # current release
         os:
           - ubuntu-latest
         arch:

Project.toml

@@ -1,13 +1,13 @@
 name = "TimeSpans"
 uuid = "bb34ddd2-327f-4c4a-bfb0-c98fc494ece1"
 authors = ["Beacon Biosignals, Inc."]
-version = "0.2.5"
+version = "0.2.6"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"
 
 [compat]
-julia = "1.0"
+julia = "1.6"
 
 [extras]
 Test = "8dfed614-e22c-5e08-85e1-65c5234f0b40"

src/TimeSpans.jl

@@ -46,7 +46,10 @@ TimeSpan(x) = TimeSpan(start(x), stop(x))
 Base.in(x::TimePeriod, y::TimeSpan) = start(y) <= x < stop(y)
 
 # work around <https://github.com/JuliaLang/julia/issues/40311>:
-Base.findall(pred::Base.Fix2{typeof(in), TimeSpan}, obj::Union{Tuple, AbstractArray}) = invoke(findall, Tuple{Function, typeof(obj)}, pred, obj)
+# written as two methods and not with obj::Union{AbstractArray,Tuple} to avoid
+# a method ambiguity in Julia 1.7
+Base.findall(pred::Base.Fix2{typeof(in), TimeSpan}, obj::AbstractArray) = invoke(findall, Tuple{Function, typeof(obj)}, pred, obj)
+Base.findall(pred::Base.Fix2{typeof(in), TimeSpan}, obj::Tuple) = invoke(findall, Tuple{Function, typeof(obj)}, pred, obj)
 
 #####
 ##### pretty printing