Skip to content

Conversation

yurishkuro
Copy link
Member

@yurishkuro yurishkuro commented Mar 29, 2025

Which problem is this PR solving?

Description of the changes

  • Move proto-gen/* under internal/

How was this change tested?

  • CI

Signed-off-by: Yuri Shkuro <github@ysh.us>
Signed-off-by: Yuri Shkuro <github@ysh.us>
@yurishkuro yurishkuro added the changelog:refactoring Internal code refactoring without functional changes label Mar 29, 2025
@yurishkuro yurishkuro requested a review from a team as a code owner March 29, 2025 18:58
@yurishkuro yurishkuro requested a review from jkowall March 29, 2025 18:58
Signed-off-by: Yuri Shkuro <github@ysh.us>
Signed-off-by: Yuri Shkuro <github@ysh.us>
Copy link

codecov bot commented Mar 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.22%. Comparing base (26bfb4f) to head (cd4e61d).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6941      +/-   ##
==========================================
- Coverage   96.26%   96.22%   -0.05%     
==========================================
  Files         343      344       +1     
  Lines       20313    20255      -58     
==========================================
- Hits        19554    19490      -64     
- Misses        574      577       +3     
- Partials      185      188       +3     
Flag Coverage Δ
badger_v1 10.21% <ø> (ø)
badger_v2 2.12% <ø> (ø)
cassandra-4.x-v1-manual 15.36% <ø> (ø)
cassandra-4.x-v2-auto 2.11% <ø> (ø)
cassandra-4.x-v2-manual 2.11% <ø> (ø)
cassandra-5.x-v1-manual 15.36% <ø> (ø)
cassandra-5.x-v2-auto 2.11% <ø> (ø)
cassandra-5.x-v2-manual 2.11% <ø> (ø)
elasticsearch-6.x-v1 20.12% <ø> (ø)
elasticsearch-7.x-v1 20.21% <ø> (ø)
elasticsearch-8.x-v1 20.38% <ø> (-0.06%) ⬇️
elasticsearch-8.x-v2 2.12% <ø> (ø)
grpc_v1 11.28% <ø> (ø)
grpc_v2 8.23% <ø> (ø)
kafka-3.x-v1 10.51% <ø> (ø)
kafka-3.x-v2 2.12% <ø> (ø)
memory_v2 2.12% <ø> (ø)
opensearch-1.x-v1 20.26% <ø> (ø)
opensearch-2.x-v1 20.26% <ø> (-0.06%) ⬇️
opensearch-2.x-v2 2.12% <ø> (ø)
tailsampling-processor 0.57% <ø> (ø)
unittests 95.00% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yurishkuro yurishkuro enabled auto-merge March 29, 2025 19:11
@yurishkuro yurishkuro added this pull request to the merge queue Mar 29, 2025
Merged via the queue into jaegertracing:main with commit 3011bd4 Mar 29, 2025
56 checks passed
@yurishkuro yurishkuro deleted the proto-gen-internal branch March 29, 2025 20:06
amilbcahat pushed a commit to amilbcahat/jaeger that referenced this pull request May 4, 2025
## Which problem is this PR solving?
- Part of jaegertracing#6869

## Description of the changes
- Move `proto-gen/*` under `internal/`

## How was this change tested?
- CI

---------

Signed-off-by: Yuri Shkuro <github@ysh.us>
amilbcahat pushed a commit to amilbcahat/jaeger that referenced this pull request May 4, 2025
## Which problem is this PR solving?
- Part of jaegertracing#6869

## Description of the changes
- Move `proto-gen/*` under `internal/`

## How was this change tested?
- CI

---------

Signed-off-by: Yuri Shkuro <github@ysh.us>
Signed-off-by: amol-verma-allen <amol.verma@allen.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:refactoring Internal code refactoring without functional changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants