Skip to content

[BAZEL] Fix version warnings in MODULE.bazel#3617

Merged
marcalff merged 1 commit intoopen-telemetry:mainfrom
marcalff:fix_bazel_warnings
Aug 30, 2025
Merged

[BAZEL] Fix version warnings in MODULE.bazel#3617
marcalff merged 1 commit intoopen-telemetry:mainfrom
marcalff:fix_bazel_warnings

Conversation

@marcalff
Copy link
Copy Markdown
Member

@marcalff marcalff commented Aug 29, 2025

Fixes # (issue)

Changes

Please provide a brief description of the changes here.

  • Adjust MODULE.bazel to list the actual version numbers seen in the dependency tree

This resolves the following build warnings:

WARNING: For repository 'bazel_skylib', the root module requires module version bazel_skylib@1.5.0, but got bazel_skylib@1.7.1 in the resolved dependency graph.
WARNING: For repository 'com_github_grpc_grpc', the root module requires module version grpc@1.63.1.bcr.1, but got grpc@1.66.0.bcr.2 in the resolved dependency graph.
WARNING: For repository 'platforms', the root module requires module version platforms@0.0.8, but got platforms@0.0.10 in the resolved dependency graph.
WARNING: For repository 'com_google_protobuf', the root module requires module version protobuf@26.0, but got protobuf@29.0 in the resolved dependency graph.
WARNING: For repository 'rules_proto', the root module requires module version rules_proto@5.3.0-21.7, but got rules_proto@7.0.2 in the resolved dependency graph.
WARNING: For repository 'com_github_google_benchmark', the root module requires module version google_benchmark@1.8.3, but got google_benchmark@1.8.4 in the resolved dependency graph.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@marcalff marcalff requested a review from a team as a code owner August 29, 2025 20:36
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.03%. Comparing base (6540d19) to head (6c346b9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3617      +/-   ##
==========================================
+ Coverage   90.02%   90.03%   +0.02%     
==========================================
  Files         220      220              
  Lines        7069     7069              
==========================================
+ Hits         6363     6364       +1     
+ Misses        706      705       -1     

see 1 file with indirect coverage changes

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

@marcalff marcalff merged commit f813f86 into open-telemetry:main Aug 30, 2025
66 checks passed
@marcalff marcalff deleted the fix_bazel_warnings branch October 18, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants