Skip to content
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.

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

Add pg-native to monorepo #3225

Merged
merged 9 commits into from
Jun 4, 2024
Merged

Add pg-native to monorepo #3225

merged 9 commits into from
Jun 4, 2024

Conversation

brianc
Copy link
Owner

@brianc brianc commented May 30, 2024

Makes fixing things like #3214 easier and generally makes contributing a bit easier. I didn't do much to "modernize" the pg-native codebase other than running it through the standard eslint --fix that is applied to the rest of the code. There's some easy opportunities there to update it to es6 and so on...it still uses some pretty antiquated coding styles in places.

Copy link

cloudflare-workers-and-pages bot commented May 30, 2024

Deploying node-postgres with  Cloudflare Pages  Cloudflare Pages

Latest commit: e1514a3
Status: ✅  Deploy successful!
Preview URL: https://878262f4.node-postgres.pages.dev
Branch Preview URL: https://bmc-add-pg-native.node-postgres.pages.dev

View logs

@brianc
Copy link
Owner Author

brianc commented Jun 3, 2024

Took me a bit to fiddle w/ the cloudflare stuff I didn't write (but now I maintain, i suppose!) - that wrangler package was doing some at-runtime npx stuff, so I upgraded it. Anyways...also re-enabled native tests for node 20, and added node 22 to the test matrix - so libpq is building & working on all nodes again. yay. Will publish this in the am (so I can be around in case there's some weird regression introduced)

@brianc brianc merged commit fe88e82 into master Jun 4, 2024
11 checks passed
@brianc brianc deleted the bmc/add-pg-native branch June 4, 2024 15:26
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.

1 participant