Skip to content

Update more deps #6057

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.

Sign up for GitHub

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

Jump to bottom
Open
congx4 wants to merge 6 commits into
base: congxie/bumpVersion2
Choose a base branch
from
Open

Update more deps #6057

congx4 wants to merge 6 commits into from

Conversation

@congx4
Copy link
Contributor

@congx4 congx4 commented Dec 26, 2025 •
edited
Loading

Description

Update more deps by running:


cargo upgrade --incompatible -p lindera-core -p lindera-dictionary -p lindera-tokenizer && cargo check --all-targets && cargo test

cargo upgrade --incompatible tikv-jemalloc-ctl -p tikv-jemallocator && cargo check --all-targets && cargo test

cargo upgrade --incompatible -p nom && cargo check --all-targets && cargo test

cargo upgrade --incompatible -p serde_json_borrow -p serde_qs && cargo check --all-targets && cargo test

How was this PR tested?


cargo check --all-targets && cargo test

make test-all

congx4
congx4 commented Dec 26, 2025
View reviewed changes
quickwit/quickwit-doc-mapper/src/routing_expression/mod.rs
pub fn wtag<'a, Error: nom::error::ParseError<&'a str>>(
t: &'a str,
) -> impl FnMut(&'a str) -> IResult<&'a str, &'a str, Error> {
) -> impl Parser<&'a str, Output = &'a str, Error = Error> {
Copy link
Contributor Author

@congx4 congx4 Dec 26, 2025

Choose a reason for hiding this comment

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

In nom 8.0, combinators return structs that implement the Parser trait, instead of returning closures.

@congx4 congx4 changed the title Congxie/bump version3 Update more deps Dec 26, 2025
@congx4 congx4 marked this pull request as ready for review December 26, 2025 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@guilload guilload Awaiting requested review from guilload

@fulmicoton-dd fulmicoton-dd Awaiting requested review from fulmicoton-dd

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@congx4