Change Details

We should avoid side-effects in the constructor as much as possible. That avoids surprising behavior at object instantiation time. The state if needed should be now built within the `swh.lister.patter.Lister.prepare` method. This also fixes the current test that actually bootstrap a real opam local "clone".

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.

We should avoid side-effects in the constructor as much as possible. That avoids surprising behavior at object instantiation time. The state if needed should be nowmust be built withininitialized into the `swh.lister.pattern.Lister.prepareget_pages` methodod, as preconized in the class docstring. This also fixes the current test that actually bootstrap a real opam local "clone". in /tmp.