Skip to content

Default pgstrap generate to PGlite#9

Open
Omar8345 wants to merge 5 commits into
seveibar:mainfrom
Omar8345:patch-1
Open

Default pgstrap generate to PGlite#9
Omar8345 wants to merge 5 commits into
seveibar:mainfrom
Omar8345:patch-1

Conversation

@Omar8345

@Omar8345 Omar8345 commented Nov 2, 2025

Copy link
Copy Markdown

Summary

  • Make pgstrap generate default to an in-memory PGlite instance so Postgres isn’t required.
  • Update CLI flag handling, docs, and tests.
  • Add a Bun helper script to run generator directly.

/claim #2
ref: #2

@Omar8345

Omar8345 commented Nov 2, 2025

Copy link
Copy Markdown
Author

@seveibar @imrishabh18, quick review appreciated 🙏

@Omar8345

Omar8345 commented Nov 8, 2025

Copy link
Copy Markdown
Author

@seveibar @imrishabh18 if you guys don't mind, i hope if you can get this reviewed today.

@imrishabh18 imrishabh18 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.

Looks okay to me, not sure if the workflow is passing or not cause this doesn't comes under the tscircuit org.

Let Seve take a look

@Omar8345

Omar8345 commented Nov 9, 2025

Copy link
Copy Markdown
Author

@seveibar sorry for the ping can you check this

@seveibar

seveibar commented Nov 9, 2025

Copy link
Copy Markdown
Owner

Hey Omar i appreciate the contrib but we're focused on user issues for tscircuit, so we dont have bandwidth to maintain our more fringe projects

@Omar8345

Omar8345 commented Nov 9, 2025

Copy link
Copy Markdown
Author

Hey Omar i appreciate the contrib but we're focused on user issues for tscircuit, so we dont have bandwidth to maintain our more fringe projects

hey Seve, I appreciate your quick reply. well no worries at all I do understand your point, either way it's a small PR so I hope it won't take much of your time 🙃

if you can get it done today, appreciate it, next week, no worries as well, but please don't get this to stale. as yk it's bounty-related

@Omar8345

Omar8345 commented Dec 1, 2025

Copy link
Copy Markdown
Author

hi again @seveibar, mind checking this please?

@Omar8345

Copy link
Copy Markdown
Author

@seveibar
cc @imrishabh18

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.

3 participants