Skip to content

docs: explain per-command environment selection - #219

Merged
Maxerns merged 1 commit into
getsotto:mainfrom
jg-noncelogic:docs/global-env-flag
Sep 11, 2026
Merged

docs: explain per-command environment selection#219
Maxerns merged 1 commit into
getsotto:mainfrom
jg-noncelogic:docs/global-env-flag

Conversation

@jg-noncelogic

Copy link
Copy Markdown
Contributor

Summary

  • document --env as a per-command environment selector in the Quick start
  • show the requested run and ls examples
  • keep all four translated READMEs structurally aligned with English

Closes #148.

Verification

  • scripts/check-readme-i18n
  • python3 -B -m unittest scripts.tests.test_readme_i18n -v (13 passed)
  • git diff --check
  • confirmed in crates/cli/src/main.rs that --env is global and overrides the configured default for the command

The full Rust suite was not run because Cargo is unavailable in this environment. The change is documentation only.

Authored with AI assistance and reviewed against the repository's contribution guidance.

Signed-off-by: jg-noncelogic <jg@noncelogic.com>
@github-actions

Copy link
Copy Markdown

Thanks for your first pull request to Sotto, and welcome.

Two things that trip up newcomers:

  • Every commit needs a Signed-off-by line under the Developer Certificate of Origin - PRs with unsigned commits cannot be merged. Sign off with git commit -s; if your branch already has unsigned commits, run git rebase --signoff main and push again.
  • Please run cargo fmt --all --check and cargo clippy --workspace --all-targets -- -D warnings before pushing, and add tests for new behaviour (cargo test --workspace runs the suite).

The full guide is in CONTRIBUTING.md - a maintainer will review shortly.

@Maxerns Maxerns left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jg-noncelogic I confirmed the per command semantics against the --env definition in main.rs. Translations across all four READMEs appreciated too. Approving ci now and ill merge when its green

@Maxerns
Maxerns merged commit 32385e7 into getsotto:main Sep 11, 2026
11 checks passed
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.

README: document the global --env flag

2 participants