jetify-com / devbox

Instant, easy, and predictable development environments
https://www.jetify.com/devbox/
Apache License 2.0
8.2k stars 192 forks source link

[gen-readme] For generated readme scripts, replace absolute paths #1949

Closed mikeland73 closed 4 months ago

mikeland73 commented 4 months ago

Summary

Plugins v2 allow scripts that may have Virtenv or DevboxDir in them. This causes the commands to show up as absolute paths in the readme. This fixes that issue.

An alternative is to read the raw scripts from the plugin (including the un-templatized string). This may be a better solution long term, but is more work and would require specially plugin parsing only for the readme generation so I punted on that approach for now.

How was it tested?

Tested with new go monorepo plugin https://github.com/jetpack-io/devbox-plugins/pull/7