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

Dependencies: all packages updated via pip-tools #10669

Merged
merged 1 commit into from Aug 28, 2023

Conversation

github-actions[bot]
Copy link
Contributor

Dependencies: all packages updated via pip-tools

@github-actions github-actions bot requested a review from a team as a code owner August 27, 2023 00:06
@github-actions github-actions bot requested a review from stsewd August 27, 2023 00:06
@humitos humitos merged commit 721104a into main Aug 28, 2023
7 checks passed
@humitos humitos deleted the dependencies/pip-tools-f999d65 branch August 28, 2023 10:59
@@ -234,7 +234,7 @@ markdown==3.4.4
# via -r requirements/pip.txt
matplotlib-inline==0.1.6
# via ipython
newrelic==8.10.1
newrelic==8.11.0
Copy link
Member

Choose a reason for hiding this comment

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

It seems there is a first party support for structlog now: https://docs.newrelic.com/docs/release-notes/agent-release-notes/python-release-notes/python-agent-81100/

We should take a closer look at NR logs after deploy.

Copy link
Member

Choose a reason for hiding this comment

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

We should look into migrating our custom integration with struclog then

Copy link
Member

Choose a reason for hiding this comment

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

Yes. I read the release notes and also the documentation but I didn't find a HowTo guide or similar. I was thinking that maybe it's "automatic" 🤔 -- If you find any documentation, please let me know or open an issue.

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.

None yet

2 participants