-
Notifications
You must be signed in to change notification settings - Fork 29.5k
Remove tool crash git.io link shortener #99574
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
Conversation
4ed7b53 to
e3be2bf
Compare
e3be2bf to
ef5d21c
Compare
| /// | ||
| /// This should only be used when the http client cannot be set directly, such as | ||
| /// when testing `package:http` code. | ||
| Future<void> overrideHttpClients(Future<void> Function() callback, FakeHttpClient httpClient) async { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found dead code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RIP git.io
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
git.io link shortener is dead, service responding 422. GitHub has removed the service as of January 2022.
Remove the code that posts a request to have the link shortened.
Fixes #53140
See also #99572
Pre-launch Checklist
///).If you need help, consider asking for advice on the #hackers-new channel on Discord.