added typemin/max for AbstractGray

[bjarthur]

see discussion here

[bjarthur]

the tests won't pass until JuliaMath/FixedPointNumbers.jl#61 is merged and tagged.

[tlnagy]

Can you close/open this to restart the tests? JuliaMath/FixedPointNumbers.jl#61 has merged so they should pass.

[bjarthur]

fixedpointnumbers needs to be tagged as well for this PR to work. given that 61 is the only commit since the last tag, it might not make sense to do so yet.

[tlnagy]

Once JuliaMath/FixedPointNumbers.jl#62 is merged, we could probably tag a new release.

[codecov-io]

Codecov Report

Merging #62 into master will increase coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
+ Coverage   64.68%   64.81%   +0.13%     
==========================================
  Files           1        1              
  Lines         269      270       +1     
==========================================
+ Hits          174      175       +1     
  Misses         95       95

Impacted Files	Coverage Δ
src/ColorVectorSpace.jl	64.81% <100%> (+0.13%)	✅

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e9b086d...e2584b5. Read the comment docs.

[bjarthur]

running tests since JuliaMath/FixedPointNumbers.jl#61 is tagged

[bjarthur]

tests are passing. will merge in a couple days unless someone has an objection.

[timholy]

Might be best to test with ===, e.g., typemin(Gray{T}) === Gray{T}(typemin(T)) or something. Otherwise I don't think we're testing that typemin(T) returns a value of type T.

[bjarthur]

changed to ===. tests pass so i'll merge in a couple days unless someone voices further concerns.

[timholy]

Thanks!