Differential D3336

swh.model.model.OriginVisit: Drop the dateutil.parser.parse use
ClosedPublic
Actions

Authored by ardumont on Jun 22 2020, 6:13 PM.

Details

Reviewers

vlorentz

Group Reviewers

Reviewers

Commits

rDMODf349bdc628bd: swh.model.model.OriginVisit: Drop the dateutil.parser.parse use

Summary

I'm planning on dropping unnecessary fields on the OriginVisit model object [1]

So it's a good time to drop this since i'll pass everywhere anyway.

This is in accordance to a discussion about phasing out dateutil.parser.parse_date in favor of using either iso-8601 datestrings or directly datetime.
Here the fields is supposed to be datetime anyway.

What do you think?

[1] D3337

Test Plan

tox

Diff Detail

Repository

rDMOD Data model

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 13034
Build 19871: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 19870: arc lint + arc unit

Event Timeline

ardumont created this revision.Jun 22 2020, 6:13 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptJun 22 2020, 6:13 PM

Build is green

Patch application report for D3336 (id=11820)

Rebasing onto ba0c4e19e6...

Current branch diff-target is up to date.

Changes applied before test

commit f349bdc628bd85a5904a3620b2cc3ca2324f6b42
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jun 22 10:14:30 2020 +0200

    swh.model.model.OriginVisit: Drop the dateutil.parser.parse use

See https://jenkins.softwareheritage.org/job/DMOD/job/tests-on-diff/75/ for more details.

Harbormaster completed remote builds in B13034: Diff 11820.Jun 22 2020, 6:15 PM

ardumont edited the summary of this revision. (Show Details)Jun 22 2020, 6:16 PM

ardumont edited the summary of this revision. (Show Details)

vlorentz accepted this revision.Jun 23 2020, 12:17 PM

This revision is now accepted and ready to land.Jun 23 2020, 12:17 PM

Closed by commit rDMODf349bdc628bd: swh.model.model.OriginVisit: Drop the dateutil.parser.parse use (authored by ardumont). · Explain WhyJun 23 2020, 5:06 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDMODf349bdc628bd: swh.model.model.OriginVisit: Drop the dateutil.parser.parse use.

ardumont mentioned this in D3345: journal: Drop datetime conversion indirection.Jun 25 2020, 9:56 AM

ardumont mentioned this in D3351: replayer: Fix isoformat datetime string for origin-visit.Jun 25 2020, 11:22 AM

ardumont mentioned this in rDSTOc6e6f33916aa: replayer: Fix isoformat datetime string for origin-visit.Jun 25 2020, 2:33 PM