as.chron methods removed. Not sure why we had them anyway.

[mattdowle]

Closes #2825

NEWS item :

as.chron methods for IDate and ITime have been removed, #2825. as.chron still works since IDate inherits from Date. We are not sure why we had specific methods in the first place. It may have been from a time when IDate did not inherit from Date, perhaps. Note that we don't use chron ourselves in our own work.

I'll rerun rev dep checks before release. If there's any impact I'll think again.

[codecov-io]

Codecov Report

Merging #2831 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2831      +/-   ##
==========================================
+ Coverage   93.45%   93.46%   +<.01%     
==========================================
  Files          61       61              
  Lines       12369    12355      -14     
==========================================
- Hits        11560    11548      -12     
+ Misses        809      807       -2

Impacted Files	Coverage Δ
R/IDateTime.R	88.63% <ø> (+0.28%)	⬆️

---

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 401967e...7b1bf80. Read the comment docs.

[jangorecki]

LGTM, another pkg less in suggests :)

[MichaelChirico]

yea the method just looks like a copy job of as.chron.Date.