chore: bump minijinja from 2.21.0 to 2.23.0 - #6197
Conversation
Bumps [minijinja](https://github.com/mitsuhiko/minijinja) from 2.21.0 to 2.23.0. - [Release notes](https://github.com/mitsuhiko/minijinja/releases) - [Changelog](https://github.com/mitsuhiko/minijinja/blob/main/CHANGELOG.md) - [Commits](mitsuhiko/minijinja@minijinja-go/v2.21.0...minijinja-go/v2.23.0) --- updated-dependencies: - dependency-name: minijinja dependency-version: 2.23.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Lock-only bump, and the one place minijinja is used is unaffected: prqlc/prqlc/src/cli/jinja.rs calls minijinja::machinery::tokenize plus Token/Span/SyntaxConfig/WhitespaceConfig, so all of 2.22.0's behaviour changes (none/bool rendering, the split filter, safety provenance, dict-method shadowing, loop-local assignments) are render-side and out of the path. cargo check -p prqlc --locked and the 5 tests in that module pass against 2.23.0 — worth checking explicitly, since the Cargo.toml comment notes the lexer sits behind unstable_machinery and isn't semver-covered.
The unrelated-looking second hunk is a resolver dedup, not a stray edit: indexmap 2.11.4 moves from hashbrown 0.15.5 to 0.16.1, which was already in the lock (arrow-array, hashlink 0.11, rsqlite-vfs). cargo metadata --locked resolves clean, and both crates declare MSRVs (1.65.0, 1.70) well under the workspace's 1.81.0.
CI state
Settled green. Every check on 0aff65a passed, including test-rust with test-dbs-external,lsp, test-msrv, test-deps-min-versions, and the final check-ok-to-merge.
The check-ok-to-merge FAILURE visible earlier belonged to run 31891407305, whose own conclusion was cancelled — the concurrency cascade making an if: always() omnibus resolve red. The replacement run re-registered it as SUCCESS.
Side note for whoever tends the bot: poll-pr-checks.sh can't verify any PR in this repo — it bails with more than 100 check contexts (exit 2), because this repo's tests matrix registers more than one page of contexts.
Bumps minijinja from 2.21.0 to 2.23.0.
Changelog
Sourced from minijinja's changelog.
Commits
19af7d4chore(release): 2.23.07f63616docs(changelog): document Python Unicode identifiers179652fref: Remove CLAUDE.md52b1cc6ref: Move claude stuff0d4c6f4ref: Remove broken symlinksbe3c6f5fix(python): enable Unicode identifiersd5bf25echore(release): 2.22.0cdbd4a9docs(changelog): document unreleased filter fixesd715d56fix: doc linkcd62d8cfix(filters): preserve safety in string transformationsMost Recent Ignore Conditions Applied to This Pull Request
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)