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

fix: remove status #325

Merged
merged 2 commits into from
Aug 8, 2024
Merged

fix: remove status #325

merged 2 commits into from
Aug 8, 2024

Conversation

aloknerurkar
Copy link
Collaborator

@aloknerurkar aloknerurkar commented Aug 8, 2024

Describe your changes

The status field is a reserved one in datadog. We should not use this tag in logs.

Issue ticket number and link

Fixes # (issue)

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation
  • I have tested this code by deploying the infrastructure and ensuring that commitments are being settled

@mrekucci mrekucci merged commit 489f0c4 into main Aug 8, 2024
5 checks passed
@mrekucci mrekucci deleted the log-improvements branch August 8, 2024 14:15
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.

3 participants