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

Align naming style of zenoh_config::defaults #1423

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

fuzzypixelz
Copy link
Member

Addresses #1304.

Although this probably won't make it into the public API anymore.

@fuzzypixelz fuzzypixelz added the internal Changes not included in the changelog label Sep 16, 2024
@eclipse-zenoh eclipse-zenoh deleted a comment from github-actions bot Sep 16, 2024
@fuzzypixelz fuzzypixelz self-assigned this Sep 16, 2024
@fuzzypixelz fuzzypixelz requested a review from wyfo September 16, 2024 14:22
@Mallets
Copy link
Member

Mallets commented Sep 17, 2024

@fuzzypixelz is this PR still valid even after having merged #1419 ?

@fuzzypixelz
Copy link
Member Author

fuzzypixelz commented Sep 18, 2024

@Mallets It's still valid but not strictly necessary as it was cut off from the (unstable) API.

@Mallets Mallets merged commit a5722dd into main Sep 18, 2024
22 of 25 checks passed
@Mallets Mallets deleted the align-zenoh-config-defaults branch September 18, 2024 12:14
Mallets added a commit that referenced this pull request Sep 19, 2024
* Fix docs

* Align naming style of `zenoh_config::defaults` (#1423)

* Put `SourceSn` behind unstable (#1448)

* Notify subscribers on `Notifier<Config>::insert_json5` (#1449)

* Hide `zenoh_buffers::reader::DidntRead` (#1447)

* Apply suggestions from code review

Co-authored-by: Mahmoud Mazouz <[email protected]>

* Add #[doc(hidden)] to internal_config feature

* Update ZBytes docs

* Update ZSerde docs

* Update ZSerde docs

* Fix ZBytesIterator docs

* Fix ZBytesWriter and ZBytesReader docs

* Fix ZBytesSliceIterator docs

* Fix ZenohId docs

* Fix docstring

* Improve FifoChannel and RingChannel docs

---------

Co-authored-by: Mahmoud Mazouz <[email protected]>
Co-authored-by: Joseph Perez <[email protected]>
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