Skip to content

(No-op) Remove duplicate imports#2420

Merged
tbavelier merged 2 commits into
mainfrom
tbavelier/duplicate-imports
Jan 6, 2026
Merged

(No-op) Remove duplicate imports#2420
tbavelier merged 2 commits into
mainfrom
tbavelier/duplicate-imports

Conversation

@tbavelier

Copy link
Copy Markdown
Member

What does this PR do?

As stated in PR title, remove duplicate imports

Motivation

Reduce number of staticcheck linter (disabled for now) violations

Additional Notes

Anything else we should know when reviewing?

Minimum Agent Versions

Are there minimum versions of the Datadog Agent and/or Cluster Agent required?

  • Agent: vX.Y.Z
  • Cluster Agent: vX.Y.Z

Describe your test plan

Write there any instructions and details you may have to test your PR.

Checklist

  • PR has at least one valid label: bug, enhancement, refactoring, documentation, tooling, and/or dependencies
  • PR has a milestone or the qa/skip-qa label

@tbavelier
tbavelier requested a review from a team as a code owner December 19, 2025 16:53
@tbavelier tbavelier added enhancement New feature or request qa/skip-qa labels Dec 19, 2025
@codecov-commenter

codecov-commenter commented Dec 19, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.28%. Comparing base (517b3d1) to head (9d3c477).
⚠️ Report is 197 commits behind head on main.

Files with missing lines Patch % Lines
...dogagentinternal/controller_reconcile_v2_common.go 0.00% 3 Missing ⚠️
...ler/datadogagent/controller_reconcile_v2_common.go 75.00% 1 Missing ⚠️
.../controller/datadogagent/feature/test/testsuite.go 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (37.50%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2420   +/-   ##
=======================================
  Coverage   37.28%   37.28%           
=======================================
  Files         290      290           
  Lines       24708    24708           
=======================================
  Hits         9212     9212           
  Misses      14783    14783           
  Partials      713      713           
Flag Coverage Δ
unittests 37.28% <37.50%> (ø)

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

Files with missing lines Coverage Δ
...ler/datadogagent/controller_reconcile_v2_common.go 30.80% <75.00%> (ø)
.../controller/datadogagent/feature/test/testsuite.go 0.00% <0.00%> (ø)
...dogagentinternal/controller_reconcile_v2_common.go 5.39% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 517b3d1...9d3c477. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tbavelier
tbavelier merged commit 4b3db1f into main Jan 6, 2026
32 checks passed
@tbavelier
tbavelier deleted the tbavelier/duplicate-imports branch January 6, 2026 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request qa/skip-qa

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants