Skip to content
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

Fix mistakenly removed links in Providers #611

Merged
merged 1 commit into from
Jun 9, 2022
Merged

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jun 9, 2022

The #610 removed too many links because GitHub throttled checking
if URLS are available.

This PR is re-done with better checking for error code.

The #610 removed too many links because GitHub throttled checking
if URLS are available.

This PR is re-done with better checking for error code.
@potiuk potiuk requested review from josh-fell and kaxil June 9, 2022 17:01
@potiuk
Copy link
Member Author

potiuk commented Jun 9, 2022

Thanks to @josh-fell for noticing those :)

@potiuk potiuk merged commit 20252de into main Jun 9, 2022
@kaxil kaxil deleted the update-missing-links branch June 9, 2022 19:30
@kaxil
Copy link
Member

kaxil commented Jun 9, 2022

Good eyes @josh-fell

potiuk added a commit that referenced this pull request Jun 17, 2023
The #610 removed too many links because GitHub throttled checking
if URLS are available.

This PR is re-done with better checking for error code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants