Differential D422

Load snapshots without depending on the structure of the previous snapshot
ClosedPublic
Actions

Authored by olasd on Sep 19 2018, 2:04 PM.

Details

Reviewers

ardumont

Commits

rDLDPYd27a7b2e4ab6: Load snapshots without depending on the structure of the previous snapshot

Summary

This makes sure we account for potential removals of versions from one visit to
the next.

Depends on D421

Test Plan

Updated the "revision unpublished" test case to match the new snapshot

Diff Detail

Repository

rDLDPY PyPI loader

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

olasd created this revision.Sep 19 2018, 2:04 PM

Harbormaster completed remote builds in B1408: Diff 1306.Sep 19 2018, 2:04 PM

Awesome!

This revision is now accepted and ready to land.Sep 19 2018, 2:41 PM

Closed by commit rDLDPYd27a7b2e4ab6: Load snapshots without depending on the structure of the previous snapshot (authored by olasd). · Explain WhySep 19 2018, 3:15 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

swh/

loader/

pypi/

loader.py

54 lines

tests/

test_loader.py

34 lines

Diff 1309

View Options

swh/loader/pypi/loader.py