Skip to content

feat: switch orchestrion to dd-trace-go v2 by default#644

Merged
eliottness merged 2 commits intomainfrom
eliottness/dd-trace-go-v2
Jul 9, 2025
Merged

feat: switch orchestrion to dd-trace-go v2 by default#644
eliottness merged 2 commits intomainfrom
eliottness/dd-trace-go-v2

Conversation

@eliottness
Copy link
Contributor

@eliottness eliottness commented Jun 12, 2025 •
edited
Loading

This PR replace all occurences of dd-trace-go.v1 with dd-trace-go/v2

@github-actions github-actions bot added the conventional-commit/chore Maintenance work that does not show up in the release notes label Jun 12, 2025
@eliottness Graphite App
Copy link
Contributor Author

eliottness commented Jun 12, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@eliottness eliottness mentioned this pull request Jun 12, 2025
Merged
@eliottness eliottness force-pushed the eliottness/dd-trace-go-v2 branch 3 times, most recently from d953577 to 05ddc3d Compare June 23, 2025 16:56
@eliottness
chore: switch orchestrion to dd-trace-go v2 by default
71bdd1a 
Signed-off-by: Eliott Bouhana <eliott.bouhana@datadoghq.com>
@eliottness eliottness force-pushed the eliottness/dd-trace-go-v2 branch from 6fb18e1 to 71bdd1a Compare July 9, 2025 12:32
@eliottness eliottness changed the title chore: switch orchestrion to dd-trace-go v2 by default feat: switch orchestrion to dd-trace-go v2 by default Jul 9, 2025
@github-actions github-actions bot added conventional-commit/feat Feature work and removed conventional-commit/chore Maintenance work that does not show up in the release notes labels Jul 9, 2025
@eliottness eliottness marked this pull request as ready for review July 9, 2025 12:35
@eliottness eliottness requested a review from a team as a code owner July 9, 2025 12:35
@eliottness
remove jnzhu/gorm from samples since it was removed from the v2 contribs
4bb1f38 
Signed-off-by: Eliott Bouhana <eliott.bouhana@datadoghq.com>
@eliottness Graphite App
Copy link
Contributor Author

eliottness commented Jul 9, 2025 •
edited
Loading

Merge activity

  • Jul 9, 1:55 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason (This repository has GitHub's merge queue enabled, which is currently incompatible with Graphite).
  • Jul 9, 1:59 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason (This repository has GitHub's merge queue enabled, which is currently incompatible with Graphite).

@eliottness eliottness added this pull request to the merge queue Jul 9, 2025
Merged via the queue into main with commit c5d7f0b Jul 9, 2025
62 checks passed
@eliottness eliottness deleted the eliottness/dd-trace-go-v2 branch July 9, 2025 14:25
@codecov
Copy link

codecov bot commented Jul 9, 2025

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 68.46%. Comparing base (1c9796d) to head (4bb1f38).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/pin/pin.go 60.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #644      +/-   ##
==========================================
+ Coverage   66.95%   68.46%   +1.51%     
==========================================
  Files         113      113              
  Lines        7916     7916              
==========================================
+ Hits         5300     5420     +120     
+ Misses       2097     1956     -141     
- Partials      519      540      +21
Components Coverage Δ
Generators 80.24% <ø> (ø)
Instruments ∅ <ø> (∅)
Go Driver 76.51% <ø> (ø)
Toolexec Driver 70.83% <ø> (ø)
Aspects 74.32% <ø> (ø)
Injector 74.87% <ø> (ø)
Job Server 68.96% <ø> (+3.04%) ⬆️
Other 68.46% <60.00%> (+1.51%) ⬆️
Files with missing lines Coverage Δ
internal/pin/pin.go 59.48% <60.00%> (+3.44%) ⬆️

... and 4 files with indirect coverage changes

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@darccio darccio darccio approved these changes

Assignees

No one assigned

Labels

conventional-commit/feat Feature work

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@eliottness @darccio