Skip to content

fix: patch 0.2.20#189

Merged
v-kessler merged 1 commit intomainfrom
vk/patch_0.2.20
Jan 29, 2026
Merged

fix: patch 0.2.20#189
v-kessler merged 1 commit intomainfrom
vk/patch_0.2.20

Conversation

@v-kessler
Copy link
Contributor

@v-kessler v-kessler commented Jan 29, 2026

  • duckdb attach catalog improvements
  • improve looping on getUser

Summary by CodeRabbit

  • Chores

    • Updated @lakekeeper/console-components dependency to the latest release.
  • Improvements

    • Enhanced the SQL Query component within warehouse pages to receive and utilize project context information, enabling more reliable query processing across projects.

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

@coderabbitai
Copy link

coderabbitai bot commented Jan 29, 2026

Walkthrough

This PR bumps the @lakekeeper/console-components dependency to v0.2.20 and adds project identifier support to the warehouse SQL query component by introducing a projectId state variable and wiring it as a new prop.

Changes

Cohort / File(s) Summary
Dependency Update
package.json
Bumps @lakekeeper/console-components from v0.2.19 to v0.2.20.
Warehouse Component Enhancement
src/pages/warehouse/[id].vue
Introduces projectId internal state populated from warehouse data (wh['project-id']) and binds it to the WarehouseSqlQuery component's new :project-id prop.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A dependency bump, so quick and clean,
A project ID passed between the scene,
The warehouse now knows which project it's in,
Small wires connected, let queries begin! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'fix: patch 0.2.20' is vague and generic, referring only to a version number without describing the actual changes being made. Consider a more descriptive title that explains what was fixed, such as 'fix: add project-id prop to WarehouseSqlQuery' or 'fix: update console-components and add project-id binding'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch vk/patch_0.2.20

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.

@v-kessler v-kessler merged commit e445a31 into main Jan 29, 2026
7 checks passed
@v-kessler v-kessler deleted the vk/patch_0.2.20 branch January 29, 2026 14:47
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