-
Notifications
You must be signed in to change notification settings - Fork 680
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
Update token_source.go #5396
Update token_source.go #5396
Conversation
minor fix Signed-off-by: Ikko Eltociear Ashimine <[email protected]>
Thank you for opening this pull request! 🙌 These tips will help get your PR across the finish line:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5396 +/- ##
=======================================
Coverage 60.99% 60.99%
=======================================
Files 794 794
Lines 51475 51475
=======================================
Hits 31398 31398
Misses 17185 17185
Partials 2892 2892
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@eltociear to. fix these CI errors, could you try rebasing to master? |
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.
Thank you!
minor fix Signed-off-by: Ikko Eltociear Ashimine <[email protected]> Co-authored-by: Kevin Su <[email protected]>
minor fix Signed-off-by: Ikko Eltociear Ashimine <[email protected]> Co-authored-by: Kevin Su <[email protected]> Signed-off-by: Vladyslav Libov <[email protected]>
Why are the changes needed?
minor fix
What changes were proposed in this pull request?
How was this patch tested?
Setup process
Screenshots
Check all the applicable boxes
Related PRs
Docs link