Skip to content

Follow diesel, sqlparser and their dependents to their main tips - #167

Merged
LucaCappelletti94 merged 1 commit into
mainfrom
r92/pin-refresh
Oct 18, 2026
Merged

LucaCappelletti94 merged 1 commit into
mainfrom
r92/pin-refresh

Conversation

@LucaCappelletti94

Copy link
Copy Markdown
Owner

Every workspace lock moves to the current main of its git dependencies. diesel now rolls back a SQLite transaction whose COMMIT fails (diesel-rs/diesel#5241), which R92's deferred reference checks rely on. sqlparser now reads a SQLite table declared WITHOUT ROWID, STRICT (apache/datafusion-sqlparser-rs#2575), which R92 parses when it rewrites the replica DDL. subql, pg2sqlite, sql-traits and sqlparser-canonicalize move with it, since each needed a change to build against the new sqlparser.

@coderabbitai

coderabbitai Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters: 7
  • Cargo.lock is excluded by !**/*.lock
  • crates/connetto-web/Cargo.lock is excluded by !**/*.lock
  • examples/dioxus-desktop-demo/Cargo.lock is excluded by !**/*.lock
  • examples/dioxus-web-demo/Cargo.lock is excluded by !**/*.lock
  • examples/wasm-smoke/Cargo.lock is excluded by !**/*.lock
  • examples/webauthn-unlock/Cargo.lock is excluded by !**/*.lock
  • examples/yew-web-demo/Cargo.lock is excluded by !**/*.lock

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration
  • Configuration used: Repository: LucaCappelletti94/coderabbit/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 19b60086-dcda-4f87-b21e-4d43122f910a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@codecov

codecov Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.78%. Comparing base (9d9feb7) to head (ff5ff4e).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main     #167    +/-   ##
========================================
  Coverage   87.78%   87.78%            
========================================
  Files         188      188            
  Lines       50190    50326   +136     
  Branches    50190    50326   +136     
========================================
+ Hits        44057    44179   +122     
- Misses       4050     4060    +10     
- Partials     2083     2087     +4     
Flag Coverage Δ
client 60.55% <ø> (-0.01%) ⬇️
rest 55.94% <ø> (+0.38%) ⬆️
server 51.81% <ø> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonarqubecloud

Copy link
Copy Markdown

@LucaCappelletti94
LucaCappelletti94 added this pull request to stack #176 October 18, 2026 08:12
@LucaCappelletti94
LucaCappelletti94 merged commit e89f61e into main Oct 18, 2026
47 of 48 checks passed
@LucaCappelletti94
LucaCappelletti94 deleted the r92/pin-refresh branch October 18, 2026 10:07
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