This conversion was irritating in my work with the volt tutorial tonight. This change is major enough to warrant a v1.1.0 release: it changes the type (beyond correcting the annotation) of a field.
The package could keep the old field around and namespace access to the new one, but I don't think v1.x.x adoption is widespread enough to merit the technical debt.
[x] Add a unit test for the conversion.
Breaking changes
List any changes that break the API usage supported on master.
Changes the type of Result.updated: time.struct_time → datetime.
Changes the type of Result.published: time.struct_time → datetime.
EPS-Libraries-Berkeley/volt#161: This came up as a pain point while migrating their example. Pandas has good support for datetimes, but not for time.struct_times.
Checklist
[x] All lint rules are satisfied: run make lint.
[x] All tests pass: run make test.
[x] All documentation is regenerated: run make docs.
[x] (If appropriate) README.md example usage has been updated.
Description
This conversion was irritating in my work with the volt tutorial tonight. This change is major enough to warrant a v1.1.0 release: it changes the type (beyond correcting the annotation) of a field.
The package could keep the old field around and namespace access to the new one, but I don't think v1.x.x adoption is widespread enough to merit the technical debt.
Breaking changes
Result.updated
:time.struct_time
→datetime
.Result.published
:time.struct_time
→datetime
.Relevant issues
datetime
s, but not fortime.struct_time
s.Checklist
make lint
.make test
.make docs
.README.md
example usage has been updated.