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
Copy link
Copy Markdown

@Omar8345 Omar8345 commented Nov 2, 2025

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
Copy link
Copy Markdown
Author

Omar8345 commented Nov 2, 2025

@seveibar @imrishabh18, quick review appreciated 🙏

@Omar8345
Copy link
Copy Markdown
Author

Omar8345 commented Nov 8, 2025

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

Copy link
Copy Markdown

@imrishabh18 imrishabh18 left a comment

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
Copy link
Copy Markdown
Author

Omar8345 commented Nov 9, 2025

@seveibar sorry for the ping can you check this

@seveibar
Copy link
Copy Markdown
Owner

seveibar commented Nov 9, 2025

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
Copy link
Copy Markdown
Author

Omar8345 commented Nov 9, 2025

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
Copy link
Copy Markdown
Author

Omar8345 commented Dec 1, 2025

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