chore: update log messages#261
chore: update log messages#261lbajolet-hashicorp merged 1 commit intovmware:mainfrom tenthirtyam:chore/update-log-messages
Conversation
lbajolet-hashicorp
left a comment
There was a problem hiding this comment.
Overall LGTM, thanks @tenthirtyam !
I left a couple comments on the changes for findFile, the updated prototype for the function looks a tad weird to me, especially with how verbose is inferred from the list of options, and only checks the first one. I'll let you address that and I'll come back after for a second pass.
Pre-approving since there's not a lot to change here
Updates log messages based on INFO or WARN context. Signed-off-by: Ryan Johnson <ryan@tenthirtyam.org>
|
@lbajolet-hashicorp - I've addressed the concerns and this one is ready again for you. |
lbajolet-hashicorp
left a comment
There was a problem hiding this comment.
LGTM, thanks for the reroll @tenthirtyam !
|
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Summary
Technical Debt: Updates log messages based on
INFOorWARNcontext.