Skip to content

fix: Node.js hybrid fixture PostgreSQL CI test#12782

Merged
NuroDev merged 2 commits intomainfrom
NuroDev/fix-nodejs-hybrid-app-pg-test
Mar 6, 2026
Merged

fix: Node.js hybrid fixture PostgreSQL CI test#12782
NuroDev merged 2 commits intomainfrom
NuroDev/fix-nodejs-hybrid-app-pg-test

Conversation

@NuroDev
Copy link
Member

@NuroDev NuroDev commented Mar 6, 2026

Fixes broken CI tests.

It seems that the RNA Central public PostgreSQL instance has made a change to make it so the reader user no longer includes the rnc_database table. We instead now use the fully qualified table name.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because: Fixes internal CI issue(s)
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: Fixes internal CI issue(s)

A picture of a cute animal (not mandatory, but encouraged)


Open with Devin

@NuroDev NuroDev self-assigned this Mar 6, 2026
@changeset-bot
Copy link

changeset-bot bot commented Mar 6, 2026

⚠️ No Changeset found

Latest commit: e84aeae

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Mar 6, 2026
@NuroDev NuroDev marked this pull request as ready for review March 6, 2026 09:13
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 6, 2026

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@12782

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@12782

miniflare

npm i https://pkg.pr.new/miniflare@12782

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@12782

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@12782

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@12782

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@12782

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@12782

wrangler

npm i https://pkg.pr.new/wrangler@12782

commit: 68ad92c

@github-project-automation github-project-automation bot moved this from Untriaged to Approved in workers-sdk Mar 6, 2026
@NuroDev NuroDev merged commit 4c7633f into main Mar 6, 2026
46 of 49 checks passed
@NuroDev NuroDev deleted the NuroDev/fix-nodejs-hybrid-app-pg-test branch March 6, 2026 09:51
@github-project-automation github-project-automation bot moved this from Approved to Done in workers-sdk Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants