Add deprecated message for conversions number to interval

[hyrodium]

This PR fixes #97.

I just removed the conversion methods. Should we add deprecation messages instead?
I guess the conversions are probably not well used.

[codecov]

Codecov Report

Merging #98 (36c1dc5) into master (4a6e746) will increase coverage by 0.04%.
The diff coverage is 100.00%.

❗ Current head 36c1dc5 differs from pull request most recent head d9ba6b9. Consider uploading reports for the commit d9ba6b9 to get more accurate results

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
+ Coverage   99.19%   99.23%   +0.04%     
==========================================
  Files           3        3              
  Lines         248      261      +13     
==========================================
+ Hits          246      259      +13     
  Misses          2        2              

Impacted Files	Coverage Δ
src/IntervalSets.jl	99.17% <100.00%> (+0.09%)	⬆️
src/interval.jl	99.10% <0.00%> (+<0.01%)	⬆️

---

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 4a6e746...d9ba6b9. Read the comment docs.

[timholy]

I would favor a deprecation, that way we don't have to bump the minor version immediately.

[dlfivefifty]

I don’t remember writing these conversions. @daanhb was it you? Any comments?

[hyrodium]

I don’t remember writing these conversions. @daanhb was it you? Any comments?

@dlfivefifty
Maybe by you? 😄

• 179646f
• Add open and mixed open/closed intervals #26 (comment)

[hyrodium]

I would favor a deprecation, that way we don't have to bump the minor version immediately.

I thought the next release will be v0.7.0 if we resolve #86 with #93. (cf #94)
I propose dropping support for 1..missing and making 1..missing throws an error.

We can release v0.6.1 with deprecation messages for the conversion of a number to an interval, but I think very few people need this deprecation.

(This is not a strong opinion and I'm okay with the deprecation messages.)

[hyrodium]

I'll update this PR with the deprecation messages (x-ref #94 (comment))

[timholy]

I like the idea of the deprecation because the standard Julia package management makes it easy to see how to fix things:

• receive a CompatHelper PR proposing to update the [compat]
• but the PR CI fails...let's make sure it really works on the current version
• run the tests, see the hint about how to fix it, and profit

[daanhb]

I don’t remember writing these conversions. @daanhb was it you? Any comments?

I don't think I will miss these conversions, at least I don't recall ever using them. In any case there are other ways of converting a number to a domain.

[hyrodium]

I updated the conversions with deprecated messages.

$ julia --depwarn=yes
               _
   _       _ _(_)_     |  Documentation: https://docs.julialang.org
  (_)     | (_) (_)    |
   _ _   _| |_  __ _   |  Type "?" for help, "]?" for Pkg help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 1.7.1 (2021-12-22)
 _/ |\__'_|_|_|\__'_|  |  Official https://julialang.org/ release
|__/                   |

julia> using IntervalSets

julia> convert(ClosedInterval, 1)
┌ Warning: `The conversion number to interval will be removed.
│   caller = top-level scope at REPL[2]:1
└ @ Core REPL[2]:1
1..1

[timholy]

Note there are quite a few others that should be changed similarly.