-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Converting Dates.FixedPeriod to Quantity{...,𝐓} #329
Conversation
I also started to implement this some time ago, I have uploaded it here so you can maybe use some of my code: https://gist.github.com/sostock/68e5eda1abee911116dcc08970843d0c The broken tests should work on Julia ≥ 1.5, since JuliaLang/julia#34645 was fixed recently. |
Co-authored-by: Sebastian Stock <42280794+sostock@users.noreply.github.com>
OK, perhaps I should close this in favour of that, you've taken this far more seriously. Would the plan be to just not run those tests on Julia < 1.5? |
Codecov Report
@@ Coverage Diff @@
## master #329 +/- ##
==========================================
- Coverage 80.23% 79.98% -0.26%
==========================================
Files 15 16 +1
Lines 1174 1194 +20
==========================================
+ Hits 942 955 +13
- Misses 232 239 +7
Continue to review full report at Codecov.
|
If you want, I can make a PR with my changes. I think I didn’t do that because I wanted to implement some more stuff (like support for
Yes, that’s how I would do it. |
OK, no great rush, but having even the basics would be nice. |
I opened #331 as a replacement. |
Fixes #125, or at least makes a start.
Edit: I moved this to a new file, as it needs to come after the relevant units have been defined.
Should things like this be made to work?