-
Notifications
You must be signed in to change notification settings - Fork 78
retries #316
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@Aslemammad Do you have plans to implement this? If not, would you give me the logs on the server side for 500 errors so that I can try implementing it? |
yes, i'm thinking of doing that in the cli package! what do you think? so any 500 error we get, we'll do a retry. @AmirSa12 can you tackle this soon? |
I think it's on the server side and not the cli. The error happens when requesting the package. |
Ah, I noticed that it can be mitigated on the ecosystem-ci side. |
amazing, so that solves your issue completely? and it does it only for the manifest as far as i know, but i guess that's enough. |
It should work in theory, but I'm not completely sure. I'll get back to you if it doesn't work. Still, having the retry on the server side is probably a good idea (though it's not a high priority). |
thank you so much! let us know. |
@sapphi-red shared this amazing idea.
The text was updated successfully, but these errors were encountered: