Skip to content
Snippets Groups Projects

Move retried_request from the toolbox

Merged Manon Blanco requested to merge request-retry-mechanism into master
All threads resolved!

Closes #38 (closed)

I added the retry mechanism directly on the request function so that the migration is transparent (we don't have to change all the client.request to client.retried_request), as we use this system everywhere (callico, base worker, cli, scrapers etc.).

Edited by Manon Blanco

Merge request reports

Pipeline #169481 passed

Pipeline passed for 239666c8 on request-retry-mechanism

Merged by Bastien AbadieBastien Abadie 10 months ago (Apr 22, 2024 11:54am UTC)

Loading

Pipeline #169980 passed

Pipeline passed for 239666c8 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Erwan Rouchet requested review from @babadie and removed review request for @erouchet

    requested review from @babadie and removed review request for @erouchet

  • Bastien Abadie resolved all threads

    resolved all threads

  • Manon Blanco mentioned in merge request !237 (merged)

    mentioned in merge request !237 (merged)

  • Please register or sign in to reply
    Loading