Skip to content

馃Ч chore: Add env var support to go module - #34

Merged
jpmcb merged 1 commit into
mainfrom
go-envvars
Aug 11, 2026
Merged

馃Ч chore: Add env var support to go module#34
jpmcb merged 1 commit into
mainfrom
go-envvars

Conversation

@jpmcb

@jpmcb jpmcb commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Adds env vars for go module

Refs PCC-1166

Signed-off-by: John McBride <john@papercompute.com>
@linear-code

linear-code Bot commented Aug 11, 2026

Copy link
Copy Markdown

PCC-1166

@greptile-apps

greptile-apps Bot commented Aug 11, 2026

Copy link
Copy Markdown

Greptile Summary

Adds optional environment-variable configuration to the Go module and applies each supplied KEY=VALUE entry to its build container.

  • Adds an envVars constructor argument and stores it on the module.
  • Validates and applies environment-variable entries when constructing the container.
  • Documents command-line and toolchain customization usage.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete actionable failures established in the changed code.

The environment variables are parsed as KEY=VALUE, applied to the shared Go container setup, and construction errors are propagated by both affected checks.

Important Files Changed

Filename Overview
go/main.go Adds optional environment-variable parsing and propagation while updating checks to handle container-construction errors.
go/README.md Documents environment-variable usage, including an unverified toolchain customization example.

Reviews (1): Last reviewed commit: "馃Ч chore: Add env var support to go modu..." | Re-trigger Greptile

@jpmcb
jpmcb merged commit 7413a24 into main Aug 11, 2026
11 checks passed
@jpmcb
jpmcb deleted the go-envvars branch August 11, 2026 21:20
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.

1 participant