Skip to content

Conversation

@tomtau
Copy link
Contributor

@tomtau tomtau commented Nov 21, 2025

Summary by CodeRabbit

  • Chores

    • Released version 2.8.4 across all core packages.
    • Updated all internal cross-package dependencies to version 2.8.4 to ensure consistency and compatibility.
  • Refactor

    • Optimized end-of-input token detection logic.

✏️ Tip: You can customize this high-level summary in your review settings.

@tomtau tomtau requested a review from a team as a code owner November 21, 2025 09:16
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 21, 2025

Walkthrough

This PR performs a coordinated version bump of the Pest ecosystem from 2.8.3 to 2.8.4 across all workspace crates (pest, debugger, derive, generator, grammars, meta, vm). All inter-crate dependencies are updated to the new version. One code change replaces an iterator check with a direct contains method.

Changes

Cohort / File(s) Summary
Version bumps and dependency updates
debugger/Cargo.toml, derive/Cargo.toml, generator/Cargo.toml, grammars/Cargo.toml, meta/Cargo.toml, pest/Cargo.toml, vm/Cargo.toml
Package versions bumped from 2.8.3 to 2.8.4; inter-crate dependencies (pest, pest_derive, pest_generator, pest_meta, pest_vm) updated to 2.8.4 across all manifests
Minor code optimization
generator/src/generator.rs
Replaced iterator .any() check with .contains(&"EOI") for cleaner EOI presence detection in defaults

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Repetitive version bumps across homogeneous Cargo.toml files reduce review complexity
  • Single code change is a straightforward refactor with no behavioral impact

Possibly related PRs

Suggested reviewers

  • NoahTheDuke

Poem

🐰 From 2.8.3 to 2.8.4 we go,
Bumping versions in a harmonic flow,
Cargo files dance in perfect sync,
While .contains() sharpens our code's ink,
One tiny tweak, seven manifests bright! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main objective of the changeset, which is a version bump from 2.8.3 to 2.8.4 across all package manifests.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 79fbb67 and dee9d91.

📒 Files selected for processing (8)
  • debugger/Cargo.toml (2 hunks)
  • derive/Cargo.toml (2 hunks)
  • generator/Cargo.toml (2 hunks)
  • generator/src/generator.rs (1 hunks)
  • grammars/Cargo.toml (2 hunks)
  • meta/Cargo.toml (2 hunks)
  • pest/Cargo.toml (1 hunks)
  • vm/Cargo.toml (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
  • GitHub Check: cargo hack check --feature-powerset
  • GitHub Check: Test Coverage
  • GitHub Check: check for semver compatibility
  • GitHub Check: check for no_std compatibility
  • GitHub Check: Minimal Versions Testing
  • GitHub Check: Unit, Style, and Lint Testing
  • GitHub Check: Documentation check
  • GitHub Check: Fuzzing
🔇 Additional comments (8)
meta/Cargo.toml (1)

4-4: LGTM! Version bump looks correct.

The package version and pest dependency are consistently updated to 2.8.4.

Also applies to: 25-25

pest/Cargo.toml (1)

4-4: LGTM! Version bump is correct.

The core pest crate version is properly updated to 2.8.4.

derive/Cargo.toml (1)

4-4: LGTM! Version and dependencies are consistent.

The package version and all pest-related dependencies are correctly updated to 2.8.4.

Also applies to: 28-29

vm/Cargo.toml (1)

4-4: LGTM! Consistent version updates.

Package and dependency versions are all correctly set to 2.8.4.

Also applies to: 17-18

grammars/Cargo.toml (1)

4-4: LGTM! Version updates are correct.

Package version and dependencies are consistently updated to 2.8.4.

Also applies to: 17-18

debugger/Cargo.toml (1)

4-4: LGTM! All versions synchronized.

Package and all pest ecosystem dependencies are correctly updated to 2.8.4.

Also applies to: 20-22

generator/Cargo.toml (1)

4-4: LGTM! Version updates are consistent.

Package version and pest-related dependencies are all correctly set to 2.8.4.

Also applies to: 25-26

generator/src/generator.rs (1)

36-36: LGTM! Cleaner implementation.

The refactor from iter().any() to contains() is more idiomatic and maintains the same behavior. Good improvement!

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

  • Provide your own instructions using the high_level_summary_instructions setting.
  • Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
  • Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

  1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
  2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
  3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
  4. 📊 Contributor Summary — Include a Markdown table showing contributions:
    | Contributor | Lines Added | Lines Removed | Files Changed |
  5. ✔️ Additional Notes — Add any extra reviewer context.
    Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tomtau tomtau merged commit 28427c6 into pest-parser:master Nov 21, 2025
9 checks passed
@tomtau tomtau deleted the bump-version-13 branch November 21, 2025 11:06
@coderabbitai coderabbitai bot mentioned this pull request Jan 3, 2026
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.

1 participant