danger / peril

☢️ Serious and immediate danger.
https://danger.systems
MIT License
460 stars 58 forks source link

Move retry logic to fetch.ts #450

Closed jtreanor closed 5 years ago

jtreanor commented 5 years ago

This is a small update to the changes I made in https://github.com/danger/peril/pull/446. It moves the logic to fetch.ts so that it is transparent to the Github helpers and applies tp anywhere fetch is called. I've also added a check that we have not exceeded the retries before throwing the error.

It more closely matches what I implemented in https://github.com/danger/danger-js/pull/898.

orta commented 5 years ago

Cool, yep!