Skip to content

feat(pin): Add version selection logic for dd-trace-go#711

Merged
kakkoyun merged 6 commits into
mainfrom
kakkoyun/orchestrion_pin_minimum_version_select
Oct 1, 2025
Merged

feat(pin): Add version selection logic for dd-trace-go#711
kakkoyun merged 6 commits into
mainfrom
kakkoyun/orchestrion_pin_minimum_version_select

Conversation

@kakkoyun

Copy link
Copy Markdown
Member

Signed-off-by: Kemal Akkoyun [email protected]

@github-actions github-actions Bot added the conventional-commit/feat Feature work label Sep 29, 2025
@kakkoyun
kakkoyun force-pushed the kakkoyun/orchestrion_pin_minimum_version_select branch 2 times, most recently from a16dfed to 9b0c6b0 Compare September 29, 2025 16:32
@kakkoyun kakkoyun changed the title feat(pin): Add minimum version selection logic feat(pin): Add version selection logic for dd-trace-go Sep 29, 2025
@kakkoyun
kakkoyun marked this pull request as ready for review September 30, 2025 13:23
@kakkoyun
kakkoyun requested a review from a team as a code owner September 30, 2025 13:23
Comment thread internal/ensure/requiredintegrations.go Outdated
Comment thread internal/pin/pin.go
@kakkoyun
kakkoyun force-pushed the kakkoyun/orchestrion_pin_minimum_version_select branch from ee5c056 to dca2c5d Compare September 30, 2025 13:45
@kakkoyun
kakkoyun force-pushed the kakkoyun/orchestrion_pin_minimum_version_select branch from 55a09c9 to 23d0a2b Compare October 1, 2025 11:41
@kakkoyun
kakkoyun force-pushed the kakkoyun/orchestrion_pin_minimum_version_select branch from 23d0a2b to 32b0b4c Compare October 1, 2025 13:02
@kakkoyun
kakkoyun marked this pull request as draft October 1, 2025 13:10
@kakkoyun
kakkoyun force-pushed the kakkoyun/orchestrion_pin_minimum_version_select branch from 32b0b4c to 68df880 Compare October 1, 2025 13:15
@kakkoyun
kakkoyun marked this pull request as ready for review October 1, 2025 13:20
@kakkoyun
kakkoyun enabled auto-merge October 1, 2025 13:20
@kakkoyun
kakkoyun requested a review from eliottness October 1, 2025 13:21
@kakkoyun
kakkoyun added this pull request to the merge queue Oct 1, 2025
@kakkoyun kakkoyun mentioned this pull request Oct 1, 2025
Merged via the queue into main with commit e2dc1b7 Oct 1, 2025
54 of 55 checks passed
@kakkoyun
kakkoyun deleted the kakkoyun/orchestrion_pin_minimum_version_select branch October 1, 2025 14:02
@codecov

codecov Bot commented Apr 3, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 70.10870% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.62%. Comparing base (e061d12) to head (68df880).
⚠️ Report is 75 commits behind head on main.

Files with missing lines Patch % Lines
internal/ensure/requiredintegrations.go 77.93% 21 Missing and 11 partials ⚠️
internal/gomod/gomod.go 32.00% 14 Missing and 3 partials ⚠️
internal/pin/pin.go 57.14% 2 Missing and 4 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #711      +/-   ##
==========================================
- Coverage   65.72%   65.62%   -0.10%     
==========================================
  Files         113      116       +3     
  Lines        7926     8477     +551     
==========================================
+ Hits         5209     5563     +354     
- Misses       2192     2364     +172     
- Partials      525      550      +25     
Components Coverage Δ
Generators 80.24% <ø> (ø)
Instruments ∅ <ø> (∅)
Go Driver 72.88% <59.40%> (-2.93%) ⬇️
Toolexec Driver 67.94% <100.00%> (+0.41%) ⬆️
Aspects 72.10% <75.32%> (+0.19%) ⬆️
Injector 72.94% <75.70%> (+0.14%) ⬆️
Job Server 65.91% <ø> (ø)
Other 65.62% <63.00%> (-0.10%) ⬇️
Files with missing lines Coverage Δ
internal/pin/pin.go 59.32% <57.14%> (-1.84%) ⬇️
internal/gomod/gomod.go 55.69% <32.00%> (ø)
internal/ensure/requiredintegrations.go 77.93% <77.93%> (ø)

... and 7 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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants