dd-trace-go icon indicating copy to clipboard operation
dd-trace-go copied to clipboard

Published 20 hours ago • verified publisher icon DataDog

WIP: feat: code origins

Open rarguelloF opened this issue 6 months ago • 1 comments

What does this PR do?

Motivation

Reviewer's Checklist

  • [ ] Changed code has unit tests for its functionality at or near 100% coverage.
  • [ ] System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
  • [ ] There is a benchmark for any new code, or changes to existing code.
  • [ ] If this interacts with the agent in a new way, a system test has been added.
  • [ ] New code is free of linting errors. You can check this by running golangci-lint run locally.
  • [ ] Add an appropriate team label so this PR gets put in the right place for the release notes.
  • [ ] Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

Unsure? Have a question? Request a review!

rarguelloF avatar May 14 '25 13:05 rarguelloF

Benchmarks

Benchmark execution time: 2025-06-11 14:40:56

Comparing candidate commit 09c68e2edcc6f2fb7bb63b6e60332cc710d54c5c in PR branch rarguelloF/LANGPLAT-387/code-origins with baseline commit ccb162b1a0a42d7db6ee2cbe53a132817d63b029 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 56 metrics, 0 unstable metrics.

pr-commenter[bot] avatar May 14 '25 14:05 pr-commenter[bot]

@rarguelloF @kakkoyun Is this PR alive?

darccio avatar Nov 14 '25 15:11 darccio

@rarguelloF @kakkoyun Is this PR alive?

We can close this one for now. It's not decided whether by whom or when this will be tackled.

Thanks a lot for the work @rarguelloF.

(We can keep the branch, in case we want to revive it, but let's close the PR)

kakkoyun avatar Nov 14 '25 15:11 kakkoyun