Update (not using a variable as this fails the build)

Hardcode the path to rsvndump which got emptied when using a variable (for some reason)

Rebase and add comment

lgtm but fix the typo in the docstring cli parameter ;)

lgtm but this is missing the migration script update first (and the bump in the schema version somewhere?).

one non blocking suggestion inline

Adapt according to remark

Just an inline remark but otherwise it's ok.

The second part looks good to me. :)

Fix build

Adapt according to first round of review

Adapt according to review

Rebase

Reword some sentences

lgtm, one question inline.

Well, wrong forge to push diff.
Fixing that in our gitlab instance [1].

Amend impacted docstrings

Adapt according to discussion

Fix mistyped signature

Add coverage (which is a bit convoluted but we are in loader-core so no real loader to
check that actual behavior beyond what i propose).

Only, more_data_to_fetch/create_snapshot is renamed create_partial_visit though as
that makes more sense now.

• Rebase
• reword commit and diff description
• adapt parameter according to review suggestion from @vlorentz

one question inline.

I wonder whether we should merge this config file and that of the other script, so that we get all settings in one place.

What do you think?

Yes, i think so too. But which script, to be sure, manage_user_groups right?

I just started it that way to simplify the reading.
I think that i'll adapt according to what you proposed here then update the diff then we land.

Add cli.py

one typo inline.

Adapt according to last remark, dump partial json updated project.

And fix some staging issues (after that first run):