elastic / beats

:tropical_fish: Beats - Lightweight shippers for Elasticsearch & Logstash
https://www.elastic.co/products/beats
Other
12.05k stars 4.89k forks source link

refactor: drop golang.org/x/tools/go/vcs dependency #40016

Open kruskall opened 3 days ago

kruskall commented 3 days ago

Proposed commit message

golang.org/x/tools/go/vcs is deprecated and was initially developed before module support was added. However we are not using vcs.Cmd at all, we're using one method to retrieve the vcs being used. As a quick fix, we can copy the method since it's short and self-contained and we don't need the whole package.

Checklist

Disruptive User Impact

Author's Checklist

How to test this PR locally

Related issues

-

Use cases

Screenshots

Logs

mergify[bot] commented 3 days ago

This pull request does not have a backport label. If this is a bug or security fix, could you label this PR @kruskall? 🙏. For such, you'll need to label your PR with:

To fixup this pull request, you need to add the backport labels for the needed branches, such as:

elasticmachine commented 3 days ago

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)