Skip to content

Conversation

@RicardoE105
Copy link
Contributor

@RicardoE105 RicardoE105 commented Jan 5, 2026

Summary

Title self explanatory

Related Linear tickets, Github issues, and Community forum posts

](https://linear.app/n8n/issue/CAT-2007/remove-conditions-where-we-check-sqliteconfigpoolsize-or)

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@RicardoE105 RicardoE105 changed the title Remove conditions where we check sqliteConfig.poolsize or isLegacySqlite choreÖ Remove conditions where we check sqliteConfig.poolsize or isLegacySqlite Jan 5, 2026
@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Jan 5, 2026
@RicardoE105 RicardoE105 changed the title choreÖ Remove conditions where we check sqliteConfig.poolsize or isLegacySqlite choreÖ Remove conditions where we check sqliteConfig.poolsize or isLegacySqlite Jan 5, 2026
@RicardoE105 RicardoE105 changed the title choreÖ Remove conditions where we check sqliteConfig.poolsize or isLegacySqlite chore: Remove conditions where we check sqliteConfig.poolsize or isLegacySqlite Jan 5, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 23 files

@codecov
Copy link

codecov bot commented Jan 5, 2026

Codecov Report

❌ Patch coverage is 50.00000% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...s/@n8n/db/src/repositories/execution.repository.ts 0.00% 4 Missing ⚠️
...ckages/@n8n/db/src/repositories/role.repository.ts 0.00% 1 Missing ⚠️
packages/cli/src/server.ts 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@currents-bot
Copy link

currents-bot bot commented Jan 5, 2026

E2E Tests: n8n tests passed after 8m 27.6s

🟢 622 · 🔴 0 · ⚪️ 27 · 🟣 4

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: 550f2af

  • Spec files: 143

  • Overall tests: 649

  • Duration: 8m 27.6s

  • Parallelization: 16

Groups

GroupId Results Spec Files Progress
multi-main:e2e 🟢 565 · 🔴 0 · ⚪️ 27 · 🟣 4 134 / 134
multi-main:e2e:isolated 🟢 57 · 🔴 0 · ⚪️ 0 9 / 9


This message was posted automatically by currents.dev | Integration Settings

@RicardoE105 RicardoE105 requested a review from ivov January 6, 2026 14:11
@RicardoE105 RicardoE105 requested a review from tomi January 23, 2026 13:19
@tomi
Copy link
Collaborator

tomi commented Jan 23, 2026

@RicardoE105 I believe this is done already in #24376

@tomi tomi closed this Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants