This breaks the current protocol, but deduplicates the code, and will allow
for the addition of extra custom encoders/decoders by callers of this module.
Details
Details
- Reviewers
olasd - Group Reviewers
Reviewers - Commits
- rDCORE6c5e89a4e029: Change msgpack serialization to be closer to the JSON one.
Diff Detail
Diff Detail
- Repository
- rDCORE Foundations and core functionalities
- Branch
- deduplicate-json-msgpack
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 10599 Build 15839: tox-on-jenkins Jenkins Build 15838: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
See https://jenkins.softwareheritage.org/job/DCORE/job/tox/408/ for more details.
Comment Actions
Have you tested remote apis with these changes? They look sound, but I guess better coverage would actually come from that.
swh/core/api/serializers.py | ||
---|---|---|
36 | I guess now that we're making a protocol break, it'd be a good time to switch over to iso8601, or at least dateutil.parser.isoparse, instead of letting ourselves parse garbage. |