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

refactor: replace message.Warn with context logger #2849

Closed
wants to merge 2 commits into from

Conversation

phillebaba
Copy link
Member

Description

This change replaces use of message.Warn with structured logging.

Related Issue

Depends on #2842
Relates to #2576

Checklist before merging

@phillebaba phillebaba changed the title Replace/warn logs refactor: replace message.Warn with context logger Aug 7, 2024
Copy link

codecov bot commented Aug 7, 2024

Signed-off-by: Philip Laine <philip.laine@gmail.com>
Copy link

netlify bot commented Aug 29, 2024

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit 6fcbe7d
🔍 Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/66d036a1294b5e00086a775c

Signed-off-by: Philip Laine <philip.laine@gmail.com>
@AustinAbro321
Copy link
Contributor

Closing as this has been superseded by the PRs linked to #2576

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants