prometheusreceiver exposes an exported function for the targetmanager package: `NewManager`. This package should move under internal or move under pkg/ if it is meant to be API.

[atoulme]

prometheusreceiver exposes an exported function for the targetmanager package: NewManager. This package should move under internal or move under pkg/ if it is meant to be API.

Originally posted by @atoulme in #43991 (comment)

[github-actions]

Pinging code owners for receiver/prometheus: @Aneurysm9 @dashpole @ArthurSens @krajorama. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triage' to set the priority and remove the needs-triage label.

[dashpole]

We should move it to an internal package

[Angith]

Hi @dashpole @ArthurSens , I would like to work on this issue. Could you please assign to me?

[ArthurSens]

Hi Angith, feel free to open PRs :)

There's no policy that only assigned people can work on issues

[tushar1977]

Hi maintainers, I noticed that my PR was submitted before @Angith’s, but it looks like some of the changes were cherry-picked into another PR while mine is still open. I just wanted to understand how the process is being handled — could you please explain the reasoning or workflow here if you dont mind?

[atoulme]

There isn't really a process if 2 people offer the same changes in a PR. Typically folks try to go by whoever chimes on the PR first, but that's not always the case. We also see a lot of activity and miss things ; I didn't see there was another PR.