Wed, Jan 20
Wed, Jan 20
vlorentz resigned from D4843: Reimplement Bitbucket lister using new Lister API.
Tue, Jan 19
Tue, Jan 19
Looks good to me, time to land this !
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Revert changes to the credentials setting method, and improve corresponding test
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Remove tenacity wait parameter
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
anlambert added inline comments to D4843: Reimplement Bitbucket lister using new Lister API.
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Remove the persistent isort conftest modification from diff
tenma updated the summary of D4843: Reimplement Bitbucket lister using new Lister API.
tenma updated the test plan for D4843: Reimplement Bitbucket lister using new Lister API.
Mon, Jan 18
Mon, Jan 18
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Have test coverage for credentials setting
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
- fix assert
- fix typing
- add a warning on credentials
Thu, Jan 14
Thu, Jan 14
vlorentz added inline comments to D4843: Reimplement Bitbucket lister using new Lister API.
tenma added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Thanks for the remarks, will go through them.
tenma updated the summary of D4843: Reimplement Bitbucket lister using new Lister API.
vlorentz added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Sorry, I clicked "submit" too early.
vlorentz requested changes to D4843: Reimplement Bitbucket lister using new Lister API.
vlorentz added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Not covered: authentication, max retries, small corner cases (see below)
tenma updated the test plan for D4843: Reimplement Bitbucket lister using new Lister API.
tenma updated the test plan for D4843: Reimplement Bitbucket lister using new Lister API.
tenma moved T2955: Port Bitbucket lister to the new Lister API from in-progress to code review on the Sprint 2021 01 board.
tenma updated the summary of D4843: Reimplement Bitbucket lister using new Lister API.
tenma updated the summary of D4843: Reimplement Bitbucket lister using new Lister API.
tenma retitled D4843: Reimplement Bitbucket lister using new Lister API from [WIP] Reimplement Bitbucket lister using new Lister API to Reimplement Bitbucket lister using new Lister API.
Wed, Jan 13
Wed, Jan 13
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Fix copyrights and rebase
anlambert changed the status of T2955: Port Bitbucket lister to the new Lister API from Open to Work in Progress.
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
anlambert retitled D4843: Reimplement Bitbucket lister using new Lister API from WIP new Bitbucket lister to [WIP] Reimplement Bitbucket lister using new Lister API.
anlambert updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Update: Remove old range lister task and update tasks tests.
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
Improved tests
ardumont added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
lgtm (minus the build failure that is ;)
Harbormaster failed remote builds in B18325: Diff 17181 for D4843: Reimplement Bitbucket lister using new Lister API!
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build has FAILED
anlambert updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
- add and improve tests
swh-public-ci added a comment to D4843: Reimplement Bitbucket lister using new Lister API.
Build is green
tenma updated the diff for D4843: Reimplement Bitbucket lister using new Lister API.
refactor auth and backoff
tenma requested review of D4843: Reimplement Bitbucket lister using new Lister API.
Mon, Jan 11
Mon, Jan 11
tenma updated the task description for T2955: Port Bitbucket lister to the new Lister API.