Page MenuHomeSoftware Heritage

tasks: normalize the url argument name of most lister

Authored by douardda on Sep 4 2019, 11:53 AM.



Since all the listing tasks accepts an url as first argument (whatever the
argument name is), it makes sense to use a simple common argument name for
this. I've chosen 'url' instead of api_baseurl/forge_url/url.

Also kill now useless new_lister() functions.

Depends on D1949

Diff Detail

rDLS Listers
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

douardda created this revision.Sep 4 2019, 11:53 AM
douardda updated this revision to Diff 6574.Sep 4 2019, 2:38 PM

change of plan: use 'url' argument everywhere

This will require a migration script in prod...

ardumont accepted this revision.Sep 4 2019, 3:01 PM
This revision is now accepted and ready to land.Sep 4 2019, 3:01 PM
douardda retitled this revision from tasks: make tasks accept positional arguments to tasks: normalize the url argument name of most lister.Sep 4 2019, 3:42 PM
douardda edited the summary of this revision. (Show Details)
This revision was automatically updated to reflect the committed changes.