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 typo #16717

Merged
merged 1 commit into from
Jan 29, 2024
Merged

Fix typo #16717

merged 1 commit into from
Jan 29, 2024

Conversation

u5surf
Copy link
Contributor

@u5surf u5surf commented Jan 28, 2024

What does this PR do?

Fix typo in nginx/metrics.py

Motivation

I'm a collaborator nginx-module-vts, while I investigated the datadog integration with vts module, I found a typo in some comments. Thus I fixed.

Additional Notes

Nothing especially.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

Copy link

github-actions bot commented Jan 28, 2024

Test Results

  4 files   4 suites   16s ⏱️
 35 tests 32 ✅  2 💤 0 ❌ 1 🔥
103 runs  90 ✅ 12 💤 0 ❌ 1 🔥

For more details on these errors, see this check.

Results for commit 738fc28.

♻️ This comment has been updated with latest results.

Copy link
Member

@FlorentClarret FlorentClarret left a comment

Choose a reason for hiding this comment

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

Thanks!

@FlorentClarret FlorentClarret merged commit 9bd72e3 into DataDog:master Jan 29, 2024
28 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants