Differential D2258

Support zero-based indexes in indexing_lister
ClosedPublic
Actions

Authored by olasd on Nov 12 2019, 5:52 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDLS1e7a905a13a4: Support zero-based indexes in indexing_lister

Summary

Looks like an avoidable limitation.

Depends on D2257

Test Plan

tox

Diff Detail

Repository

rDLS Listers

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 8965
Build 13099: tox-on-jenkins	Jenkins
Build 13098: arc lint + arc unit

Event Timeline

olasd created this revision.Nov 12 2019, 5:52 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptNov 12 2019, 5:52 PM

olasd added a child revision: D2260: indexing lister: Force bounds of integer ranges to be integers.Nov 12 2019, 5:54 PM

Build is green
See https://jenkins.softwareheritage.org/job/DLS/job/tox/486/ for more details.

Harbormaster completed remote builds in B8965: Diff 7788.Nov 12 2019, 5:56 PM

ardumont accepted this revision.Nov 12 2019, 6:49 PM

This revision is now accepted and ready to land.Nov 12 2019, 6:49 PM

Closed by commit rDLS1e7a905a13a4: Support zero-based indexes in indexing_lister (authored by olasd). · Explain WhyNov 12 2019, 7:55 PM

This revision was automatically updated to reflect the committed changes.

olasd added a commit: rDLS1e7a905a13a4: Support zero-based indexes in indexing_lister.

Revision Contents
Changeset List

Path

Size

swh/

lister/

core/

indexing_lister.py

2 lines

tests/

test_indexing_lister.py

22 lines

Diff 7788

View Options

swh/lister/core/indexing_lister.py