Skip to content

Future-proof against some upcoming fetcher changes. #237

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.

Sign up for GitHub

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

Jump to bottom
Open
mwyman wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Future-proof against some upcoming fetcher changes. #237

mwyman wants to merge 5 commits into from

Conversation

@mwyman
Copy link

@mwyman mwyman commented Oct 11, 2022

To eliminate dispatch_semaphore usage, the GTMSessionFetcherService protocol was changed recently; this adds the required method to ensure the tests don't break if building with the latest fetcher.
petea
petea reviewed Oct 11, 2022
View reviewed changes
@mwyman
Replace @synthesize of GTMSessionFetcherService protocol methods wi…
00365c4 
…th explicit implementations.

This future-proofs against fetcher changes.
@mwyman mwyman changed the title Add empty method to satisfy fetcher service protocol changes. Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants

@mwyman @petea @alaoy0225