Upgrade #5
Merged
Upgrade #5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Enable 1 more harness. * Upgrade Kani version. * Fix version
* Don't ignore errors in prost_attrs * Update prost-derive to syn 2 * Bump msrv to 1.63 --------- Co-authored-by: Lucio Franco <[email protected]>
Co-authored-by: Lucio Franco <[email protected]>
Signed-off-by: Alexander Seiler <[email protected]> Co-authored-by: Lucio Franco <[email protected]>
Depend on itertools >=0.10, <0.12 Co-authored-by: Lucio Franco <[email protected]>
…s#891) The `Message` derive macro currently generates `Debug` implementations that contain code that violates the `non_snake_case` linter rule[^1]. This commit changes the macro expansion to add an `#[allow(...)]` attribute for this rule. [^1]: https://doc.rust-lang.org/stable/nightly-rustc/rustc_lint/nonstandard_style/static.NON_SNAKE_CASE.html Co-authored-by: Lucio Franco <[email protected]>
* chore: Fix yaml indent * chore: Replace arduino/setup-protoc with taiki-e/install-action * chore: Remove unused git submodule * chore: Remove outdated disabled ci job * chore: Refactor ci job step name * chore: Replace seanmiddleditch/gha-setup-ninja with in-tree action * chore: Change ci name short --------- Co-authored-by: Lucio Franco <[email protected]>
…ts (tokio-rs#851) * feat(prost-build): do not escape brackets followed by parenthesis or bracket in comments * feat(prost-build): do not escape already escaped brackets in doc comments * fix(boostrap-test): only escape brackets once --------- Co-authored-by: Lucio Franco <[email protected]>
Co-authored-by: Lucio Franco <[email protected]>
* Make Debug impl optional on derive(Message) * Config to optionally skip debug for messages * Inline things again * Put more things behind if * Optionally skip debug in oneofs too * Optionally skip debug for all types * fixup! Optionally skip debug for all types * Fix merge * Remove superfluous push_indent * Add some tests for the macro property * Fix test * Test generated protobuf * Unconditional Debug * Import --------- Co-authored-by: Lucio Franco <[email protected]>
* feat: `Name` trait + `Any` encoding support As discussed in tokio-rs#299 and tokio-rs#858, adds a `Name` trait which associates a type name and package constants with a `Message` type. It also provides `full_name` and `type_url` methods. The `type_url` method is used by newly added methods on the `Any` type which can be used for decoding/encoding messages: - `Any::from_msg`: encodes a given `Message`, returning `Any`. - `Any::to_msg`: decodes `Any::value` as the given `Message`, first validating the message type has the expected type URL. * Add private `TypeUrl` type Implements the basic rules for parsing type URLs as documented in: https://github.com/protocolbuffers/protobuf/blob/a281c13/src/google/protobuf/any.proto#L129C2-L156C50 Notably this extracts the final path segment of the URL which contains the full name of the type, and uses that for type comparisons. * CI: bump test toolchain to 1.64 This is the MSRV of `petgraph` now: error: package `petgraph v0.6.4` cannot be built because it requires rustc 1.64 or newer, while the currently active rustc version is 1.63.0 * Add `Name` impls for well-known protobuf types Also adds tests for `Any::{from_msg, to_msg}`. * Fix no_std --------- Co-authored-by: Lucio Franco <[email protected]>
# Conflicts: # .github/workflows/ci.yml # Cargo.toml # prost-build/src/lib.rs # prost-derive/Cargo.toml # prost-derive/src/field/mod.rs # prost-derive/src/lib.rs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.