Skip to content

refactor: refactor envbuilder to use coder/serpent as CLI engine #140

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

Merged
merged 22 commits into from
Apr 29, 2024
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
15e00e2
Refactor options structure
BrunoQuaresma Apr 24, 2024
f723785
Merge branch 'main' of github.com:coder/envbuilder into bq/docs
BrunoQuaresma Apr 24, 2024
68d4523
Fix broken internal test
BrunoQuaresma Apr 24, 2024
1cdf3f3
Remove Dependencies struct and pass deps directly as arg
BrunoQuaresma Apr 24, 2024
ff1b0d0
Remove unecessary arg
BrunoQuaresma Apr 24, 2024
8a626ed
Migrate to user serpent
BrunoQuaresma Apr 24, 2024
247514f
Fix 'Init Args description should end with a period'
BrunoQuaresma Apr 24, 2024
c8cbdf5
Fix others 'should end with a period' errors
BrunoQuaresma Apr 24, 2024
816d8a3
Merge branch 'main' of github.com:coder/envbuilder into bq/docs
BrunoQuaresma Apr 25, 2024
e94b8c3
Fix config.json
BrunoQuaresma Apr 25, 2024
f03fa6e
Adjust description and flags
BrunoQuaresma Apr 25, 2024
9cd9367
Use options back to reduce PR changes
BrunoQuaresma Apr 25, 2024
babf6b8
Use logf back to reduce PR changes
BrunoQuaresma Apr 25, 2024
a383098
Remove Hidden
BrunoQuaresma Apr 25, 2024
f11f3f3
Document exported values on options.go
BrunoQuaresma Apr 25, 2024
e3b205b
Rename c to Options in test
BrunoQuaresma Apr 25, 2024
9b5ca70
Use shorter name for variable
BrunoQuaresma Apr 26, 2024
bb53a5a
Fix descriptions
BrunoQuaresma Apr 26, 2024
2e5aa71
Nit var declaration
BrunoQuaresma Apr 26, 2024
d67ade5
Fix git config json removal
BrunoQuaresma Apr 26, 2024
866853a
Add filesystem and logger back to Options
BrunoQuaresma Apr 26, 2024
ce4bb56
Fix WorkspaceFolder assignment
BrunoQuaresma Apr 26, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix others 'should end with a period' errors
  • Loading branch information
BrunoQuaresma committed Apr 24, 2024
commit c8cbdf56cfedd981cc77170e37d3f6252ca0f40d
12 changes: 6 additions & 6 deletions config.go
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ func (c *Config) Options() serpent.OptionSet {
Description: `ForceSafe ignores any filesystem safety checks.
This could cause serious harm to your system!
This is used in cases where bypass is needed
to unblock customers!`,
to unblock customers.`,
},
{
Name: "Insecure",
Expand Down Expand Up @@ -207,7 +207,7 @@ func (c *Config) Options() serpent.OptionSet {
Env: "GIT_URL",
Value: serpent.StringOf(&c.GitURL),
Description: `GitURL is the URL of the Git repository to clone.
This is optional!`,
This is optional.`,
},
{
Name: "Git Clone Depth",
Expand All @@ -228,28 +228,28 @@ func (c *Config) Options() serpent.OptionSet {
Env: "GIT_USERNAME",
Value: serpent.StringOf(&c.GitUsername),
Description: `GitUsername is the username to use for Git authentication.
This is optional!`,
This is optional.`,
},
{
Name: "Git Password",
Env: "GIT_PASSWORD",
Value: serpent.StringOf(&c.GitPassword),
Description: `GitPassword is the password to use for Git authentication.
This is optional!`,
This is optional.`,
},
{
Name: "Git HTTP Proxy URL",
Env: "GIT_HTTP_PROXY_URL",
Value: serpent.StringOf(&c.GitHTTPProxyURL),
Description: `GitHTTPProxyURL is the url for the http proxy.
This is optional!`,
This is optional.`,
},
{
Name: "Workspace Folder",
Env: "WORKSPACE_FOLDER",
Value: serpent.StringOf(&c.WorkspaceFolder),
Description: `WorkspaceFolder is the path to the workspace folder
that will be built. This is optional!`,
that will be built. This is optional.`,
},
{
Name: "SSL Cert Base64",
Expand Down