8000 Fix uninitialized instance variable by jasonmp85 · Pull Request #20558 · DataDog/integrations-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix uninitialized instance variable #20558

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 20, 2025
Merged

Fix uninitialized instance variable #20558

merged 1 commit into from
Jun 20, 2025

Conversation

jasonmp85
Copy link
Contributor

Seems to have been missed, just an initialization line.

What does this PR do?

Fixes an uninitialized variable error.

Motivation

A colleague created a work item.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@jasonmp85
Fix uninitialized instance variable
7bcc2d8 
Seems to have been missed, just an initialization line.
@jasonmp85 jasonmp85 added the qa/skip-qa Automatically skip this PR for the next QA label Jun 20, 2025
@jasonmp85 jasonmp85 requested review from a team as code owners June 20, 2025 17:00
@codecov Codecov
Copy link
codecov bot commented Jun 20, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.26%. Comparing base (5eafaa7) to head (7bcc2d8).

Additional details and impacted files
Flag Coverage Δ
activemq ?
cassandra ?
confluent_platform ?
hive ?
hivemq ?
hudi ?
ignite ?
jboss_wildfly ?
kafka ?
postgres 93.14% <100.00%> (+3.50%) ⬆️
presto ?
solr ?
tomcat ?
weblogic ?

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jasonmp85 jasonmp85 added this pull request to the merge queue Jun 20, 2025
Merged via the queue into master with commit 383b5fd Jun 20, 2025
48 of 53 checks passed
@jasonmp85 jasonmp85 deleted the DBMON-5468 branch June 20, 2025 18:41
github-actions bot pushed a commit to bskinny/integrations-core that referenced this pull request Jun 21, 2025
@bskinny
Fix uninitialized instance variable (DataDog#20558)
bdb1faa 
Seems to have been missed, just an initialization line. 383b5fd
Kyle-Neale pushed a commit that referenced this pull request Jun 24, 2025
@jasonmp85 @Kyle-Neale
Fix uninitialized instance variable (#20558)
0d8d3c7 
Seems to have been missed, just an initialization line.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amw-zero amw-zero amw-zero approved these changes

@azhou-datadog azhou-datadog azhou-datadog approved these changes

Assignees
No one assigned
Labels
agent/review-requested changelog/no-changelog ecosystems/review-requested integration/postgres product/review-requested qa/skip-qa Automatically skip this PR for the next QA team/agent-integrations team/database-monitoring-agent
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jasonmp85 @amw-zero @azhou-datadog
0