Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue/api sync #61

Merged
merged 2 commits into from
Mar 21, 2024
Merged

Issue/api sync #61

merged 2 commits into from
Mar 21, 2024

Conversation

DariusIMP
Copy link
Member

Updating Kotlin API

In this PR we remove the publisher.write(value, sampleKind) function from the API since it was removed on Zenoh Rust.

Fixes #60

@DariusIMP
Copy link
Member Author

@fuzzypixelz Can you please review? 😄

Copy link
Member

@fuzzypixelz fuzzypixelz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Mallets Mallets merged commit 0de23e6 into eclipse-zenoh:main Mar 21, 2024
7 checks passed
@Mallets Mallets deleted the issue/api-sync branch March 21, 2024 13:15
Mallets pushed a commit that referenced this pull request Apr 15, 2024
* feat(examples arguments): Adding CLI args to ZSub.

* feat(examples arguments): Adding CLI args to ZPub.

* feat(examples arguments): Adding CLI args to ZGet.

* feat(examples arguments): Adding CLI args to ZPut.

* feat(examples arguments): Adding attachment argument for ZGet.

* feat(examples arguments): CLI arguments for ZDelete.

* feat(examples arguments): CLI arguments for ZPubThr.

* feat(examples arguments): CLI arguments for ZSubThr.

* feat(examples arguments): CLI arguments for ZQueryable.

* feat(examples arguments): tidying up code

* feat(examples arguments): updating README

* Add the missing trait

* Cargo fmt

* Update Cargo.lock to include the fix of binding interface

* Typo

* Trigger CI

* Issue/api sync (#61)

* Adapt JNI codebase to recent Zenoh changes.

* issue(api sync): removing publisher.write() after it was removed from the Rust API

* feat(examples arguments): Adding CLI args to ZSub.

* feat(examples arguments): Adding CLI args to ZPub.

* feat(examples arguments): Adding CLI args to ZGet.

* feat(examples arguments): Adding CLI args to ZPut.

* feat(examples arguments): Adding attachment argument for ZGet.

* feat(examples arguments): CLI arguments for ZDelete.

* feat(examples arguments): CLI arguments for ZPubThr.

* feat(examples arguments): CLI arguments for ZSubThr.

* feat(examples arguments): CLI arguments for ZQueryable.

* feat(examples arguments): tidying up code

* feat(examples arguments): updating README

* Add the missing trait

* Cargo fmt

* Typo

* Trigger CI

* Removing unused import

* feat(examples arguments): replacing multiple arguments format for the listen and connect arguments.

For instance:

  gradle ZPub --args="-l tcp/localhost:7447 tcp/localhost:7448 tcp/localhost:7449"

now becomes

  gradle ZPub --args="-l tcp/localhost:7447 -l tcp/localhost:7448 -l tcp/localhost:7449"

---------

Co-authored-by: yuanyuyuan <[email protected]>
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.

Remove publisher write
3 participants