gpt-engineer icon indicating copy to clipboard operation
gpt-engineer copied to clipboard

Reference pipx and BASE_URL in readme.md

Open NoCLin opened this issue 1 year ago • 3 comments

NoCLin avatar Jun 21 '23 15:06 NoCLin

PTAL @AntonOsika @patillacode @carlthome

NoCLin avatar Jun 21 '23 15:06 NoCLin

Thanks for the idea!

Def don't want to add additional bloat to readme and make us look like projects like smol-dev though

  1. What problem does the BASE_URL solve specifically? Do you have examples?
  2. Regarding pipx, any drawbacks? Should we just delete all references to venv in Makefile, and only refer to pipx for developers? (and say pip and pipx works for stable release)

AntonOsika avatar Jun 25 '23 18:06 AntonOsika

Regarding pipx, any drawbacks? Should we just delete all references to venv in Makefile, and only refer to pipx for developers? (and say pip and pipx works for stable release)

pipx isn't for development. It's a convenience tool for installing applications for usage without having to fiddle with virtual environments (pipx installs into global but isolated venvs by default). venv is still beneficial for editable installs for package development.

I agree this might be considered bloat. Users of pipx are probably power users, and it's a popular preference but not an obvious standard (yet?).

carlthome avatar Jun 25 '23 19:06 carlthome

Stale -> closing for now!

AntonOsika avatar Jun 29 '23 20:06 AntonOsika