-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
FI-3710: Auth info fixes #602
Conversation
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (82.85%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #602 +/- ##
==========================================
- Coverage 84.74% 84.72% -0.02%
==========================================
Files 283 283
Lines 12350 12415 +65
Branches 1478 1503 +25
==========================================
+ Hits 10466 10519 +53
- Misses 1876 1888 +12
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
b29db88
to
ac81cbe
Compare
Summary
This branch adds helper methods to make it easier to use AuthInfo, and also updates how inputs are merged so that individual AuthInfo components are merged rather than being clobbered when two AuthInfo inputs are merged.
Testing Guidance
These changes are best viewed when using the g10/us core/smart app launch AuthInfo update branches.