flagon-io/hello

public

Greets people from the command line. The sample repository for g1t.

Document --lang in the README usage section #55

Mergedg1tbot merged into maing1t-agenton g1t

Approved by g1t-agent

ResolvedDocument --lang in the README usage section#50

Merged into main by g1t automatically, once it was ready, · e58de96

syntaqxopened this pull request with g1t-agent

I documented --lang in README.md and only touched that file.

  • Arguments table: I added a --lang LANG row. It lists en, es and fr as the supported values and en as the default.
  • Examples: I added the two blocks from the issue, --lang es Ana and --lang fr Luc. I put them after the --times example and before --version, in the same style as the existing ones.
  • Troubleshooting: I added an entry for the unknown-language error. It explains the exit status 2 and that stderr names en, es and fr.

I ran cargo run -- --lang es Ana and cargo run -- --lang fr Luc. They printed ¡Hola, Ana! and Bonjour, Luc !. I also ran --lang de X, which exited with status 2 and printed error: --lang value must be one of en, es, fr, got "de". That matches the new Troubleshooting entry. I did not run grep -q -- '--lang' README.md, but the file now contains --lang.

PRs #7 and #38 also edit README.md. My additions are small and local, but I did not check whether they conflict with those PRs. I did not open any issues or leave comments on other PRs.

syntaqxmarked this ready for review·
g1tbotrequested a review from g1t-agent·
g1t-agentapproved these changes

README-only change. I ran --lang es Ana, --lang fr Luc and --lang de X. The output matches the new examples, and the error text and exit status 2 match the Troubleshooting entry and src/lib.rs. The table row (en/es/fr, default en) is accurate. --lang is in README.md, so the grep check passes. I found no problems.

Reviewed by a g1t agent on Claude Sonnet 5.5.

g1tbotadded this to the merge queue·
g1tbotmerged this·

Sign in to comment.