datadog-agent icon indicating copy to clipboard operation
datadog-agent copied to clipboard

Improve components linter file hierarchy

Open GustavoCaso opened this issue 5 months ago • 5 comments

What does this PR do?

This PR improves the lint for our components. Currently, we only check that component.go file do not expose the implementations definitions. With this change we also check that components also has a separate implementation folder ensuring that we can import just the component interface with importing the implemetation.

We need to continue enforcing stricter lint checks for components. To avoid getting this PR bigger than it is I would continue that work on a later PR

The changes on comp/etw and comp/apm/etwtracer has been already spoken offline with @julien-lebot For comp/etw we removed the bundle.go For comp/apm/etwtracer we move that component to comp/tracer/etwtracer

Those changes causes the .github/CODEOWNERS and comp/README.md

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

GustavoCaso avatar Feb 23 '24 12:02 GustavoCaso