Fix Depreciation

[Nosferican]

No description provided.

[nalimilan]

Thanks, but that's not correct without using Compat (and bumping the requirement to the version that introduced that feature). The fact that tests passed indicates that this function isn't actually tested.

[Nosferican]

Compat 23570. Going to leave the version here for later. Better wait until the function gets a test?

[nalimilan]

Better fix it even if we don't have a test, though adding a test would be even better.

[codecov]

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   97.77%   97.77%           
=======================================
  Files           9        9           
  Lines         673      673           
=======================================
  Hits          658      658           
  Misses         15       15

Impacted Files	Coverage Δ
src/value.jl	94.11% <100%> (ø)	⬆️

---

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 e1c57a2...d1c25d1. Read the comment docs.

[nalimilan]

You also need to add using Compat somewhere, e.g. to src/CategoricalArrays.jl.

[Nosferican]

I think the Require file had dropped Compat, but src/CategoricalArrays.jl still had it in line 14.

[nalimilan]

Thanks!