Skip to content

fix(lint): remove elidedable lifetimes.#330

Merged
BewareMyPower merged 1 commit intomasterfrom
fix.lint
Feb 18, 2025
Merged

fix(lint): remove elidedable lifetimes.#330
BewareMyPower merged 1 commit intomasterfrom
fix.lint

Conversation

@mattisonchao
Copy link
Member

Motivation

Fix the clippy validation.

error: the following explicit lifetimes could be elided: 'a
    --> src/producer.rs:1181:6
     |
[11](https://github.com/streamnative/pulsar-rs/actions/runs/13383814710/job/37376880167?pr=329#step:5:12)81 | impl<'a, T: SerializeMessage + Sized, Exe: Executor> MessageBuilder<'a, T, Exe> {
     |      ^^                                                             ^^
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_lifetimes
     = note: `-D clippy::needless-lifetimes` implied by `-D warnings`
     = help: to override `-D warnings` add `#[allow(clippy::needless_lifetimes)]`
help: elide the lifetimes
     |
1[18](https://github.com/streamnative/pulsar-rs/actions/runs/13383814710/job/37376880167?pr=329#step:5:19)1 - impl<'a, T: SerializeMessage + Sized, Exe: Executor> MessageBuilder<'a, T, Exe> {
1181 + impl<T: SerializeMessage + Sized, Exe: Executor> MessageBuilder<'_, T, Exe> {
     |

error: could not compile `pulsar` (lib test) due to 1 previous error

https://github.com/streamnative/pulsar-rs/actions/runs/13391632216/job/37400455409

@BewareMyPower BewareMyPower merged commit 1e12590 into master Feb 18, 2025
7 checks passed
@BewareMyPower BewareMyPower deleted the fix.lint branch February 18, 2025 14:16
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