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

fix: clarify vault error messages #60

Merged
merged 1 commit into from
Feb 2, 2024
Merged

fix: clarify vault error messages #60

merged 1 commit into from
Feb 2, 2024

Conversation

mattnichols
Copy link
Collaborator

@mattnichols mattnichols commented Feb 2, 2024 •
edited
Loading

This adds clarity to vault exception messages so it is more apparent where the exception was raised.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

gerfboy
gerfboy approved these changes Feb 2, 2024
View reviewed changes
Copy link

@gerfboy gerfboy left a comment

Choose a reason for hiding this comment

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

snazzy

@mattnichols mattnichols merged commit d3e9d7f into master Feb 2, 2024
4 checks passed
@mattnichols mattnichols deleted the matt/clarify_vault_exception branch February 2, 2024 23:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@gerfboy gerfboy gerfboy 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
@mattnichols @gerfboy