Skip to content
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

Jump to bottom

Added new error types for CredentialsManagerException #783

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Added new error types for CredentialsManagerException #783

merged 2 commits into from
Dec 6, 2024

Conversation

pmathew92
Copy link
Contributor

Changes

Added two new error types to the CredentialsManagerException class .

Testing

  • This change adds unit test coverage

  • This change has been tested on the latest version of the platform/language or why not

Checklist

@pmathew92 pmathew92 requested a review from a team as a code owner December 3, 2024 07:26
arpit-jn
arpit-jn approved these changes Dec 6, 2024
View reviewed changes
Copy link

@arpit-jn arpit-jn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pmathew92 pmathew92 merged commit 1676b16 into main Dec 6, 2024
8 checks passed
@pmathew92 pmathew92 deleted the SDK-5200 branch December 6, 2024 06:50
@pmathew92 pmathew92 mentioned this pull request Dec 6, 2024
Merged
@skjolber skjolber mentioned this pull request Jan 24, 2025
Closed
6 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@arpit-jn arpit-jn arpit-jn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pmathew92 @arpit-jn