Skip to content

opam: Move the state initialization into the get_pages method

Antoine R. Dumont requested to merge generated-differential-D6310-source into generated-differential-D6310-target

We should avoid side-effects in the constructor as much as possible. That avoids surprising behavior at object instantiation time. The state if needed must be initialized into the swh.lister.pattern.Lister.get_pages method, as preconized in the class docstring.

This also fixes the current test that actually bootstrap a real opam local "clone" in /tmp.

Test Plan

tox (happy)

Migrated from D6310 (view on Phabricator)

Merge request reports