Skip to content

feat(deno): add postgresjs integration#22448

Open
isaacs wants to merge 1 commit into
isaacs/deno-auto-ingest-orchestrion-integrationsfrom
isaacs/deno-orchestrion-integrations-postgresjs
Open

feat(deno): add postgresjs integration#22448
isaacs wants to merge 1 commit into
isaacs/deno-auto-ingest-orchestrion-integrationsfrom
isaacs/deno-orchestrion-integrations-postgresjs

Conversation

@isaacs

@isaacs isaacs commented Jul 21, 2026

Copy link
Copy Markdown
Member

Also, remove a needlessly repetitive copypasta comment from the Deno integration tests. We know that they're integration tests, it helps no one to have a comment stating that it tests the integration.

Re: JS-2634

Comment thread packages/deno/src/sdk.ts
@isaacs
isaacs force-pushed the isaacs/deno-auto-ingest-orchestrion-integrations branch from 70b742f to 8b22101 Compare July 21, 2026 17:45
@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from 4fa00c3 to 78d4ca0 Compare July 21, 2026 17:45
@isaacs
isaacs marked this pull request as ready for review July 21, 2026 17:45
@isaacs
isaacs requested a review from a team as a code owner July 21, 2026 17:45
@isaacs
isaacs requested review from JPeer264 and andreiborza and removed request for a team July 21, 2026 17:45
Comment thread dev-packages/deno-integration-tests/suites/orchestrion-postgresjs/test.ts Outdated
@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from 78d4ca0 to e78bd19 Compare July 21, 2026 17:54

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit e78bd19. Configure here.

@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from e78bd19 to a7a4c77 Compare July 21, 2026 18:04
@isaacs
isaacs force-pushed the isaacs/deno-auto-ingest-orchestrion-integrations branch from 8b22101 to 1425bbe Compare July 21, 2026 18:04
@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from a7a4c77 to 0a6149f Compare July 21, 2026 18:20
@isaacs
isaacs requested review from a team as code owners July 21, 2026 18:20
@isaacs
isaacs requested review from chargome, logaretm, msonnb and nicohrubec and removed request for a team July 21, 2026 18:20
@isaacs
isaacs force-pushed the isaacs/deno-auto-ingest-orchestrion-integrations branch from 1425bbe to 06a6843 Compare July 21, 2026 18:20
@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from 0a6149f to f79742a Compare July 21, 2026 19:36
@isaacs
isaacs force-pushed the isaacs/deno-auto-ingest-orchestrion-integrations branch from 06a6843 to efc6182 Compare July 21, 2026 19:36
@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from f79742a to efbd0e7 Compare July 21, 2026 19:49
Also, remove a needlessly repetitive copypasta comment from the Deno
integration tests. We know that they're integration tests, it helps no
one to have a comment stating that it tests the integration.
@isaacs
isaacs force-pushed the isaacs/deno-orchestrion-integrations-postgresjs branch from efbd0e7 to 48bbe5f Compare July 21, 2026 21:43
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.

2 participants