Conversation
Fix "go install" flag for building a Go program with given build tags: --tags => -tags
Greptile SummaryCorrects a documentation typo in the Go backend: Confidence Score: 5/5Safe to merge — single-line documentation correction with no code changes. The change is a one-word typo fix in docs that matches the actual runtime behavior in the Rust source. No logic, config, or test files are touched. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["mise.toml\n[tools] tags option"] --> B["GoBackend::install_version_"]
B --> C["cmd.arg('-tags').arg(tags)"]
C --> D["go install -tags <tags> <module>@<version>"]
D --> E["Binary installed to GOBIN"]
Reviews (1): Last reviewed commit: "fix(docs): typo in Go Backend" | Re-trigger Greptile |
There was a problem hiding this comment.
Code Review
This pull request corrects a typo in the Go backend documentation, changing the incorrect --tags flag to the correct -tags flag for go install. The review feedback suggests further enhancing the documentation by capitalizing "Go" and clarifying that multiple tags should be comma-separated.
| ### `tags` | ||
|
|
||
| Specify go build tags (passed as `go install --tags`): | ||
| Specify go build tags (passed as `go install -tags`): |
There was a problem hiding this comment.
While fixing the flag typo, consider capitalizing "Go" as it refers to the language name. Additionally, clarifying that multiple tags should be comma-separated would be helpful for users.
| Specify go build tags (passed as `go install -tags`): | |
| Specify Go build tags (comma-separated, passed as go install -tags): |
Fix "go install" flag for building a Go program with given build tags: --tags => -tags
### 🐛 Bug Fixes - **(docs)** typo in Go Backend by @dolmen in [#9065](#9065) - **(task)** render dependency templates even when no args are passed by @MatthiasGrandl in [#9062](#9062) - support npm semver ranges in devEngines by @risu729 in [#9061](#9061) ### New Contributors - @dolmen made their first contribution in [#9065](#9065) - @MatthiasGrandl made their first contribution in [#9062](#9062)
Fix "go install" flag for building a Go program with given build tags: --tags => -tags