Skip to content

fix(pg): add verify option to PoolConfig#75264

Open
Abhi-DevHub wants to merge 1 commit into
DefinitelyTyped:masterfrom
Abhi-DevHub:fix/pg-poolconfig-verify
Open

fix(pg): add verify option to PoolConfig#75264
Abhi-DevHub wants to merge 1 commit into
DefinitelyTyped:masterfrom
Abhi-DevHub:fix/pg-poolconfig-verify

Conversation

@Abhi-DevHub

Copy link
Copy Markdown

Closes #74971

Adds the verify option to PoolConfig in @types/pg. The underlying pg-pool package supports this option for connection verification. The type follows the signature from pg-pool source (lines 347-361).

Also adds a test case using the verify callback.

Signed-off-by: Abhishek Hiremath <131762197+Abhi-DevHub@users.noreply.github.com>
@typescript-automation

typescript-automation Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

@Abhi-DevHub Thank you for submitting this PR!

This is a live comment that I will keep updated.

1 package in this PR

Code Reviews

Because this is a widely-used package, a DT maintainer will need to review it before it can be merged.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ✅ Continuous integration tests have passed
  • 🕐 Most recent commit is approved by a DT maintainer

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 75264,
  "author": "Abhi-DevHub",
  "headCommitOid": "8aee0c881eadf48685bf7a7d740e75d24177d92c",
  "mergeBaseOid": "f415145935160b3b950cd29e1d3f8b7b190204c2",
  "lastPushDate": "2026-07-16T12:29:41.000Z",
  "lastActivityDate": "2026-07-16T12:29:41.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "tooManyCommits": false,
  "tooManyReviews": false,
  "popularityLevel": "Critical",
  "pkgInfo": [
    {
      "name": "pg",
      "kind": "edit",
      "files": [
        {
          "path": "types/pg/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/pg/pg-tests.ts",
          "kind": "test"
        }
      ],
      "owners": [
        "pspeter3"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Critical"
    }
  ],
  "reviews": [],
  "mainBotCommentID": 4991828892,
  "ciResult": "pass"
}

@typescript-automation

Copy link
Copy Markdown
Contributor

🔔 @pspeter3 — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

@typescript-automation typescript-automation Bot moved this to Waiting for Code Reviews in Pull Request Status Board Jul 16, 2026
@typescript-automation typescript-automation Bot moved this from Waiting for Code Reviews to Needs Maintainer Review in Pull Request Status Board Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Needs Maintainer Review

Development

Successfully merging this pull request may close these issues.

pg: PoolConfig missing verify option from pg-pool

1 participant