-
Notifications
You must be signed in to change notification settings - Fork 3
Add support for Application default credentials #42
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
IgorRosocha
left a comment
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.
👋 Cocoapods, good job!
| add(to: &request) | ||
| } | ||
| } | ||
|
|
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.
❕ nitpick: redundant empty line
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.
Fixed 8412b20
* Prepare for App default credentials * Update GoogleAuth lib * Update CI * Make tests compilable * Fix sorted keys in tests * Remove Cocoapods support * Update changelog * Remove pods from README * Add ADC to README * Remove unused TokenProvider extension * Remove extra line
Add support for Application default credentials, this means changing from previously used and unmaintained Google authentication lib, to my implementation.
Also needed to update some CI stuff and removed Cocoapods support.