Skip to content

chore: remove version for zenoh-test#2621

Merged
fuzzypixelz merged 1 commit into
eclipse-zenoh:mainfrom
ZettaScaleLabs:update-zenoh-test-dep
May 26, 2026
Merged

chore: remove version for zenoh-test#2621
fuzzypixelz merged 1 commit into
eclipse-zenoh:mainfrom
ZettaScaleLabs:update-zenoh-test-dep

Conversation

@diogomatsubara

@diogomatsubara diogomatsubara commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

Fix release.yml workflow by setting zenoh-test as a path only dependency.

What does this PR do?

Remove version key from zenoh-test workspace definition

  • zenoh-test is non-publishable and dev-dependency only.
  • when the version key is defined it takes precedence over the path dependency during publication.

Why is this change needed?

This change avoids the error seen in https://github.com/eclipse-zenoh/zenoh/actions/runs/26425305854/job/77845882170

Related Issues

#2573
eclipse-zenoh/ci#451


🏷️ Label-Based Checklist

Based on the labels applied to this PR, please complete these additional requirements:

Labels: internal

🏠 Internal Change

This PR is marked as internal (not user-facing):

  • No API changes - Public APIs unchanged
  • No behavior changes - External behavior identical
  • Refactoring/maintenance - Code improvements only
  • Tests still pass - All existing tests pass without modification

Lighter review: Internal changes may have lighter review requirements.

Instructions:

  1. Check off items as you complete them (change - [ ] to - [x])
  2. The PR checklist CI will verify these are completed

This checklist updates automatically when labels change, but preserves your checked boxes.

@diogomatsubara
diogomatsubara marked this pull request as ready for review May 26, 2026 11:11
@diogomatsubara
diogomatsubara force-pushed the update-zenoh-test-dep branch from 81fa157 to 20d79ce Compare May 26, 2026 11:28
@diogomatsubara
diogomatsubara force-pushed the update-zenoh-test-dep branch from 20d79ce to 32acc2d Compare May 26, 2026 13:42
@diogomatsubara diogomatsubara added the internal Changes not included in the changelog label May 26, 2026
@codecov

codecov Bot commented May 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.02%. Comparing base (692a0c7) to head (32acc2d).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2621      +/-   ##
==========================================
- Coverage   74.07%   74.02%   -0.06%     
==========================================
  Files         400      400              
  Lines       61044    61030      -14     
==========================================
- Hits        45218    45175      -43     
- Misses      15826    15855      +29     

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

@fuzzypixelz
fuzzypixelz merged commit a6d75a0 into eclipse-zenoh:main May 26, 2026
34 of 35 checks passed
@evshary
evshary deleted the update-zenoh-test-dep branch May 27, 2026 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Changes not included in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants