Fix round.IDate behavior for "weeks"

[MichaelChirico]

Re-push of #4334. Related issue: #4335

I plan to follow this up with further improvements to this method, which seems pretty broken to me at the moment.

[MichaelChirico]

• add week_start parameter to round.IDate #6025
• Fix round.IDate behavior for "weeks" #6024 👈
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MichaelChirico and the rest of your teammates on Graphite

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.53%. Comparing base (566bff0) to head (7f767a0).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6024   +/-   ##
=======================================
  Coverage   97.53%   97.53%           
=======================================
  Files          80       80           
  Lines       14915    14915           
=======================================
  Hits        14547    14547           
  Misses        368      368           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[MichaelChirico]

I might hold off on this pending #6025 -- if we don't have a week_start parameter then I find round.Date()/round.IDate() to be pretty hopeless and would push for deprecation in favor of trunc.IDate().