Skip to content

build: prepare Raven 2.26.2 - #900

Merged
martian56 merged 1 commit into
mainfrom
release/2.26.2
Sep 12, 2026
Merged

martian56 merged 1 commit into
mainfrom
release/2.26.2

Conversation

@martian56

@martian56 martian56 commented Sep 12, 2026 •

Copy link
Copy Markdown
Owner

Patch release: the String indexing segfault fix (#894, #896) and the dedicated binary operator diagnostic with operator-aware conversion hints (#895, #897). Version bump in Cargo.toml with Cargo.lock refreshed, CHANGELOG entry added.

Summary by CodeRabbit

  • Bug Fixes

    • Indexing a String now produces a clear compiler error instead of causing a runtime crash.
    • Invalid binary operations now show operator-specific diagnostics, including accepted types and relevant suggestions.
  • Chores

    • Updated the release version to 2.26.2.

@martian56 martian56 self-assigned this Sep 12, 2026
@coderabbitai

coderabbitai Bot commented Sep 12, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: fa3514e2-1073-4253-98fc-f687db3026c1

📥 Commits

Reviewing files that changed from the base of the PR and between 4583a23 and a875c22.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • CHANGELOG.md
  • Cargo.toml

Included review availability: Your plan provides up to 2 included reviews per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The release version changes to 2.26.2. The changelog documents fixes for String indexing errors and binary operator diagnostics.

Changes

Compiler fixes release

Layer / File(s) Summary
Version and changelog update
Cargo.toml, CHANGELOG.md
The workspace version changes from 2.26.1 to 2.26.2. The changelog documents the String indexing and binary operator diagnostic fixes.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other · Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to a875c

The release metadata is consistent and introduces no identified merge-blocking risk.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description accurately summarizes the release changes, but it omits the required template sections for Changes, Test plan, Related issues, and Notes for reviewers. Use the repository template. Add the required headings, list the changes, document the test plan and completed checks, identify related issues such as #894, #895, #896, and #897, and add reviewer notes or state that none apply.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the primary change: preparing Raven version 2.26.2. It is concise and specific.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release/2.26.2

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.

@martian56
martian56 merged commit d1645b5 into main Sep 12, 2026
6 checks passed
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