Consume the iterator returned by tool_add.

Test Plan

Unfortunately, this file has no proper test, see #1393 (closed). I'm going to send a diff to swh.deposit that cover this, though.

---

Migrated from D718 (view on Phabricator)

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/24/ See console output for more information: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/24/console

mentioned in commit swh-deposit@d17dff96

Please explain why you need to consume the iterator in your commit message or as a comment in the code.

Merge request was returned for changes

• Explain in the commit message why list() is needed.

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/29/ See console output for more information: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/29/console

mentioned in merge request swh-deposit!14 (closed)

Merge request was accepted

approved this merge request

• git magic

landed

Merge request was merged

closed

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/32/ See console output for more information: https://jenkins.softwareheritage.org/job/DLDBASE/job/tox/32/console