Skip to content

Bump crate versions #295

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

jprendes
Copy link
Contributor

@jprendes jprendes commented Apr 9, 2025

This PR bumps the crate versions for publishing.
The compiler crate needs a major version bump after breaking changes introduced by PR #291
The codegen crate needs a major version bump because it re-exports ttrpc_compiler::Customize.
The main crate needs a major version bump after the breaking changes introduced by PR #290

This PR builds on top of #294: Diff excluding the changes in #294
I'll rebase once #294 is merged

@Tim-Zhang
Copy link
Member

Hi, @jprendes. There are commits which has already been merged in the PR, please remove them, thanks.

jprendes added 5 commits May 20, 2025 20:52
Add a cargo workspace that links all the crates in the project together.
This should make publishing easier when "package-workspace" stabilises.

Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
Skip running rustfmt on the code generated by the examples

Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
add publish = false to ttrpc-example to avoid publishing it by mistake

Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
Add RELEASE.md with release intructions

Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
Bump crate versions for publishing.
The compiler crate needs a major version bump after breaking changes introduced by PR containerd#291
The codegen crate needs a major version bump because it exports an item from the compiler crate.
The main crate needs a major version bump after the braking changes introduced by PR containerd#290

Signed-off-by: Jorge Prendes <jorge.prendes@gmail.com>
@jprendes
Copy link
Contributor Author

rebased, but this PR is still waiting for #294 to be merged.

@jprendes
Copy link
Contributor Author

There's an unrelated test failure

thread 'main' panicked at example/./async-client.rs:38:9:
assertion `left == right` failed
  left: Err(RpcStatus(Status { code: DEADLINE_EXCEEDED, message: "timeout", details: [], special_fields: SpecialFields { unknown_fields: UnknownFields { fields: None }, cached_size: CachedSize { size: 0 } } }))
 right: Err(Others("Receive packet timeout Elapsed(())"))
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Looks like 2 slightly different versions of the same error.

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