Skip to content

chore(deps): pin embedded-postgres to beta.15 patch #4102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
yuanLiang2 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

chore(deps): pin embedded-postgres to beta.15 patch #4102

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion cli/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"drizzle-orm": "0.38.4",
"dotenv": "^17.0.1",
"commander": "^13.1.0",
"embedded-postgres": "^18.1.0-beta.16",
"embedded-postgres": "18.1.0-beta.15",
"picocolors": "^1.1.1"
},
"devDependencies": {
Expand Down
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@
"packageManager": "pnpm@9.15.4",
"pnpm": {
"patchedDependencies": {
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot Apr 20, 2026

Choose a reason for hiding this comment

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

P2 PR description is incomplete

The PR description still contains unfilled template placeholders in the Thinking Path ([Which subsystem or capability is involved], [What problem or gap exists], etc.), and all checklist items are unchecked. The "What Changed", "Verification", "Risks", and "Model Used" sections only contain -. Per CONTRIBUTING.md, a complete thinking path, change rationale, verification steps, and risk assessment are required before merge.

Context Used: CONTRIBUTING.md has a guide for a good PR message ... (source)

Prompt To Fix With AI 
This is a comment left during a code review.
Path: package.json
Line: 55

Comment:
**PR description is incomplete**

The PR description still contains unfilled template placeholders in the Thinking Path (`[Which subsystem or capability is involved]`, `[What problem or gap exists]`, etc.), and all checklist items are unchecked. The "What Changed", "Verification", "Risks", and "Model Used" sections only contain `-`. Per CONTRIBUTING.md, a complete thinking path, change rationale, verification steps, and risk assessment are required before merge.

**Context Used:** CONTRIBUTING.md has a guide for a good PR message ... ([source](https://app.greptile.com/review/custom-context?memory=instruction-0))

How can I resolve this? If you propose a fix, please make it concise.

"embedded-postgres@18.1.0-beta.16": "patches/embedded-postgres@18.1.0-beta.16.patch"
"embedded-postgres@18.1.0-beta.15": "patches/embedded-postgres@18.1.0-beta.15.patch"
},
"overrides": {
"rollup": ">=4.59.0"
Expand Down
2 changes: 1 addition & 1 deletion packages/db/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
"dependencies": {
"@paperclipai/shared": "workspace:*",
"drizzle-orm": "^0.38.4",
"embedded-postgres": "^18.1.0-beta.16",
"embedded-postgres": "18.1.0-beta.15",
"postgres": "^3.4.5"
},
"devDependencies": {
Expand Down
4 changes: 2 additions & 2 deletions ...es/embedded-postgres@18.1.0-beta.16.patch → ...es/embedded-postgres@18.1.0-beta.15.patch
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ diff --git a/dist/index.js b/dist/index.js
// The default configuration options for the class
const defaults = {
databaseDir: path.join(process.cwd(), 'data', 'db'),
@@ -133,7 +133,7 @@
@@ -121,7 +121,7 @@
`--pwfile=${passwordFile}`,
`--lc-messages=${LC_MESSAGES_LOCALE}`,
...this.options.initdbFlags,
Expand All @@ -19,7 +19,7 @@ diff --git a/dist/index.js b/dist/index.js
// Connect to stderr, as that is where the messages get sent
(_a = process.stdout) === null || _a === void 0 ? void 0 : _a.on('data', (chunk) => {
// Parse the data as a string and log it
@@ -177,7 +177,7 @@
@@ -165,7 +165,7 @@
'-p',
this.options.port.toString(),
...this.options.postgresFlags,
Expand Down
Loading
Loading